[PATCH] D30979: [GlobalISel][X86] G_LOAD/G_STORE, add alternative RegisterBank mapping.

Zvi Rackover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 26 05:59:54 PDT 2017


zvi accepted this revision.
zvi added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/Target/X86/X86RegisterBankInfo.cpp:124
+    const MachineInstr &MI,
+    const SmallVectorImpl<PartialMappingIdx> &OpRegBankIdx,
+    SmallVectorImpl<const ValueMapping *> &OpdsMapping) {
----------------
Consider passing an ArrayRef instead


Repository:
  rL LLVM

https://reviews.llvm.org/D30979





More information about the llvm-commits mailing list