[patch] Use "set -o pipefail"

Stephen Lin swlin at post.harvard.edu
Thu Jul 4 19:01:49 PDT 2013


On Thu, Jul 4, 2013 at 3:07 PM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
>> It seems somewhat dangerous, though, since a test that is meant to
>> exit with a non-zero status could start crashing instead without
>> causing any test failures (depending on what other checks were in the
>> pipeline).
>
> Good point.
>
>> Maybe this should be another utility instead? It could even be the
>> same binary but symlinked and have the behavior depend on the name...
>
> I have added a --crash option to not instead. New patches for llvm and
> clang are attached.

Great, thanks!

>
>> Stephen
>
> Cheers,
> Rafael




More information about the llvm-commits mailing list