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