[PATCH] D78609: [llvm] [X86] Add "REQUIRES asserts" to test
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 21 22:08:58 PDT 2020
craig.topper added a comment.
In D78609#1995999 <https://reviews.llvm.org/D78609#1995999>, @mehdi_amini wrote:
> I don't really understand why? Can you add the motivation in the commit/revision description?
I think because the test is checking the -debug output.
Can we just check the assembly output and trust that asserts builds will trigger the assert that found this originally?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78609/new/
https://reviews.llvm.org/D78609
More information about the llvm-commits
mailing list