RKSimon added a comment. In D135938#3867170 <https://reviews.llvm.org/D135938#3867170>, @FreddyYe wrote:
> I get your point of "close to each other" and updated. And I merged the > Disasm tests, while I didn't merge the MC tests because it is not so > convenient to do. See latest updated. > > Do we need to rename old tests to follow this rule? Old tests: > https://github.com/llvm/llvm-project/tree/main/llvm/test/MC/X86 and > https://github.com/llvm/llvm-project/tree/main/llvm/test/MC/Disassembler/X86 Its not a priority, but if you are ever bored and want to do some cleaning then it help! 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