[PATCH] D107542: [M68k][GloballSel] RegBankSelect implementation

Sushma Unnibhavi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 9 00:16:04 PDT 2021


sushmaunnibhavi added inline comments.


================
Comment at: llvm/lib/Target/M68k/GlSel/M68kRegisterBankInfo.cpp:96
+  case G_STORE: {
+    OperandsMapping = &M68k::ValueMappings[M68k::GPR3OpsIdx];
+    break;
----------------
myhsu wrote:
> Why line 86, 92, and 96 are the same? Can we combine them?
Yes it can be combined.


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

https://reviews.llvm.org/D107542



More information about the llvm-commits mailing list