diff options
author | Chandler Carruth <chandlerc@gmail.com> | 2012-07-02 20:43:21 +0000 |
---|---|---|
committer | Chandler Carruth <chandlerc@gmail.com> | 2012-07-02 20:43:21 +0000 |
commit | 1f7ebddd5f6f6787023a0b7b2cd3dd4e80e10447 (patch) | |
tree | 41857536ca7f1323972e9fc2b7db64d637e6cc82 | |
parent | 20d8c1645c84a8d946fc21b599ebf49e3e798ddd (diff) | |
download | external_llvm-1f7ebddd5f6f6787023a0b7b2cd3dd4e80e10447.zip external_llvm-1f7ebddd5f6f6787023a0b7b2cd3dd4e80e10447.tar.gz external_llvm-1f7ebddd5f6f6787023a0b7b2cd3dd4e80e10447.tar.bz2 |
Revert r159528 which taught lit's builtin shell test runner about the
'|&' bash syntax. We have lots of users with a bash on their system
which doesn't support this syntax, and as bash is still significantly
faster, we should support them.
The test suite has already been updated to cope with this.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@159580 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | utils/lit/lit/ShUtil.py | 19 |
1 files changed, 4 insertions, 15 deletions
diff --git a/utils/lit/lit/ShUtil.py b/utils/lit/lit/ShUtil.py index 0c5bdac..dda622a 100644 --- a/utils/lit/lit/ShUtil.py +++ b/utils/lit/lit/ShUtil.py @@ -134,8 +134,6 @@ class ShLexer: if c == '|': if self.maybe_eat('|'): return ('||',) - if self.maybe_eat('&'): - return ('|&',) return (c,) if c == '&': if self.maybe_eat('&'): @@ -207,7 +205,7 @@ class ShParser: # Otherwise see if it is a terminator. assert isinstance(tok, tuple) - if tok[0] in ('|','|&',';','&','||','&&'): + if tok[0] in ('|',';','&','||','&&'): break # Otherwise it must be a redirection. @@ -226,18 +224,9 @@ class ShParser: negate = True commands = [self.parse_command()] - while 1: - tok = self.look() - if tok == ('|',): - self.lex() - commands.append(self.parse_command()) - continue - if tok == ('|&',): - self.lex() - commands[-1].redirects.insert(0, (('>&',2),'1')) - commands.append(self.parse_command()) - continue - break + while self.look() == ('|',): + self.lex() + commands.append(self.parse_command()) return Pipeline(commands, negate) def parse(self): |