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 | 498525e4bbd5fdf0ddaa58f04a7d00ae8e4e81fb (patch) | |
tree | 02c5c87c1629b04072574db929baf1c4cee18773 /llvm/utils/lit | |
parent | b1cdde7d21b704482a6ed2a32dbee635d7677ad3 (diff) | |
download | bcm5719-llvm-498525e4bbd5fdf0ddaa58f04a7d00ae8e4e81fb.tar.gz bcm5719-llvm-498525e4bbd5fdf0ddaa58f04a7d00ae8e4e81fb.zip |
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.
llvm-svn: 159580
Diffstat (limited to 'llvm/utils/lit')
-rw-r--r-- | llvm/utils/lit/lit/ShUtil.py | 19 |
1 files changed, 4 insertions, 15 deletions
diff --git a/llvm/utils/lit/lit/ShUtil.py b/llvm/utils/lit/lit/ShUtil.py index 0c5bdac408a..dda622a48a8 100644 --- a/llvm/utils/lit/lit/ShUtil.py +++ b/llvm/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): |