[PATCH] D54328: Added 'cd -' built in to lit

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 13 13:34:32 PST 2018


rnk added a comment.

In https://reviews.llvm.org/D54328#1294801, @joerg wrote:

> I'm quite against this. "cd -" is a non-portable extension to shell and should never be used in first place.


Can you elaborate? In what way is `cd -` non-portable? Is it a bash-ism? If it is, why should lit care? Lit looks for "bash" by name on every platform, so far as I know.


Repository:
  rL LLVM

https://reviews.llvm.org/D54328





More information about the llvm-commits mailing list