MaskRay added inline comments.

================
Comment at: llvm/test/MC/X86/intel-syntax-branch-fail.s:1
+// RUN: not llvm-mc -triple i686-unknown-unknown -x86-asm-syntax=intel %s 2>&1 
| FileCheck %s
+
----------------
Don't create a separate test for negative tests. Use the `--defsym=ERR=1` and 
`.ifdef ERR` pattern to keep positive and negative tests in one file.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149579/new/

https://reviews.llvm.org/D149579

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to