[PATCH] D67104: GlobalISel/TableGen: Fix handling of EXTRACT_SUBREG constraints

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 09:51:50 PDT 2019


paquette added inline comments.


================
Comment at: utils/TableGen/GlobalISelEmitter.cpp:3977-3978
+    //
+    // FIXME: This may introduce an extra copy if the chosen class doesn't
+    //        actually contain the subregisters.
+    const auto &SrcRCDstRCPair =
----------------
Shouldn't `getMatchingSubClassWithSubRegs` always return a class which contains the subregisters?

If `SuperClass` doesn't have an appropriate subclass, then `getMatchingSubClassWithSubRegs` should return None, no?


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

https://reviews.llvm.org/D67104





More information about the llvm-commits mailing list