<div dir="ltr"><div>For the lit shell issues, yes, you can see them on non-Windows by setting LIT_USE_INTERNAL_SHELL=1 in the environment. There was some desire in the past to standardize on the lit internal shell so that everyone sees the same failures, but I think more uses of advanced shell features have been added to test suites that don't run on Windows (tsan, msan). That makes it harder to motivate the migration. You can also search for REQUIRES: shell to find some.</div><div><br></div><div>For the quoting and path issues, I don't have a great solution other than running tests on Windows. This failure was particularly hard to detect because it has to do with the colon character, which separates flags in *SAN_OPTIONS environment variables. Do the pre-merge checks run tests on Windows yet? I know that is a goal, but I don't know the status of it. GMail is telling me that Christian Kuhnel is OOO.</div><div><br></div><div>As far as uploading to phab to trigger test runs goes, I think it auto-ccs llvm-commits and triggers herald rules, which could be noisy. I think in the long run we might be able to trigger test runs from branches uploaded to personal github repos cloned from llvm-project. For now I don't see a good way to catch these kinds of portability issues pre-commit, so we'll just live with it. =/</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 4, 2020 at 10:42 AM Vedant Kumar <<a href="mailto:vedant_kumar@apple.com">vedant_kumar@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I'm sorry about the breakage.<br>
<br>
Is there a mode I can enable when testing with lit to suss out this type of failure pre-commit? Or, if I should just create a Phab review each time and let the pre-merge checks catch issues, that would be fine too.<br>
<br>
vedant<br>
<br>
> On Mar 3, 2020, at 1:19 PM, Reid Kleckner via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> <br>
> <br>
> Author: Reid Kleckner<br>
> Date: 2020-03-03T13:18:58-08:00<br>
> New Revision: 5d3a9959382762f84318b52f6ba6532cb8b5c807<br>
> <br>
> URL: <a href="https://github.com/llvm/llvm-project/commit/5d3a9959382762f84318b52f6ba6532cb8b5c807" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/5d3a9959382762f84318b52f6ba6532cb8b5c807</a><br>
> DIFF: <a href="https://github.com/llvm/llvm-project/commit/5d3a9959382762f84318b52f6ba6532cb8b5c807.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/5d3a9959382762f84318b52f6ba6532cb8b5c807.diff</a><br>
> <br>
> LOG: Port ubsan nullability.c test to Windows<br>
> <br>
> Two issues:<br>
> - Need to add env prefix to be compatible with the lit internal shell<br>
> - Need to quote the colon in Windows paths with '"%t.supp"'<br>
> <br>
> Added: <br>
> <br>
> <br>
> Modified: <br>
>    compiler-rt/test/ubsan/TestCases/Misc/nullability.c<br>
> <br>
> Removed: <br>
> <br>
> <br>
> <br>
> ################################################################################<br>
> diff  --git a/compiler-rt/test/ubsan/TestCases/Misc/nullability.c b/compiler-rt/test/ubsan/TestCases/Misc/nullability.c<br>
> index 50295fe503f9..a68d0f8ea362 100644<br>
> --- a/compiler-rt/test/ubsan/TestCases/Misc/nullability.c<br>
> +++ b/compiler-rt/test/ubsan/TestCases/Misc/nullability.c<br>
> @@ -6,7 +6,7 @@<br>
> // RUN: echo "nullability-arg:nullability.c" > %t.supp<br>
> // RUN: echo "nullability-return:nullability.c" >> %t.supp<br>
> // RUN: echo "nullability-assign:nullability.c" >> %t.supp<br>
> -// RUN: UBSAN_OPTIONS=suppressions=%t.supp %run %t 2>&1 | FileCheck -allow-empty -check-prefix=SUPPRESS %s<br>
> +// RUN: env UBSAN_OPTIONS=suppressions='"%t.supp"' %run %t 2>&1 | FileCheck -allow-empty -check-prefix=SUPPRESS %s<br>
> // SUPPRESS-NOT: runtime error<br>
> <br>
> // CHECK: nullability.c:[[@LINE+2]]:41: runtime error: null pointer returned from function declared to never return null<br>
> <br>
> <br>
> <br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote></div>