[PATCH] D79785: [ARM] Register pressure with -mthumb forces register reload before each call

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 26 13:07:15 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:2236
+        // FIXME: How to obtain number of available registers ?
+        // Hardcoded for now.
+        if (F.arg_size() + CallInstruction->getNumOperands() >= 7)
----------------
I wrote a check for the number of arguments in a different way at https://reviews.llvm.org/D49465 ; maybe you can borrow that?  Checking the number of operands in the IR is a very inaccurate approximation.


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

https://reviews.llvm.org/D79785



More information about the llvm-commits mailing list