FreddyYe marked 2 inline comments as done.
FreddyYe added inline comments.

================
Comment at: llvm/test/MC/Disassembler/X86/avx-vnni_int8-att.txt:1
+# RUN: llvm-mc --disassemble %s -triple=i686 | FileCheck %s
+
----------------
RKSimon wrote:
> I think we'd be better off merging the att/intel test files and using 
> --check-prefix 
I see many old tests split them into two files. What about remove 32bit att 
test and 64 bit intel test, it can also help reduce code base?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135938

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

Reply via email to