[PATCH] D150667: [GlobalIsel][X86] Legalize G_BSWAP

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 07:09:34 PDT 2023


arsenm added inline comments.


================
Comment at: llvm/test/CodeGen/X86/GlobalISel/legalize-bswap.mir:8-17
+alignment:       16
+legalized:       false
+regBankSelected: false
+registers:
+  - { id: 0, class: _, preferred-register: '' }
+  - { id: 1, class: _, preferred-register: '' }
+liveins:
----------------
tschuett wrote:
> arsenm wrote:
> > Don't need any of this in any legalize tests 
> It comes from the checks script.
The checks script don't modify the tests themselves. We have really bad MIRPrinter defaults and you don't need any of this header


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150667



More information about the llvm-commits mailing list