[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