[PATCH] D136203: [ARM] Support -mexecute-only with -mlong-calls.

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 19 09:59:49 PDT 2022


efriedma added a comment.

> I am thinking about adding a new option, say -mgot-calls to allow code generation with the extra indirection. Is it sensible and shall I create another diff to discuss that?

That probably makes sense, yes.



================
Comment at: llvm/lib/Target/ARM/ARMISelLowering.cpp:2655
+      // support movw movt, so we are safe to assume that.
+      if (Subtarget->genExecuteOnly()) {
+        SDValue GA = DAG.getTargetGlobalAddress(GVal, dl, PtrVt);
----------------
Can we directly check that movw/movt is available?  I think that's what we do in other places?  (Then just assert we aren't execute-only in the non-movw path.)


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

https://reviews.llvm.org/D136203



More information about the cfe-commits mailing list