[PATCH] D67147: Remove the additional constant which requires an extra register for statepoint lowering.

zuojian lin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 4 18:57:23 PDT 2019


linzj marked an inline comment as done.
linzj added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/StatepointLowering.cpp:865-867
+      ActualCallee =
+          DAG.getIntPtrConstant(ConstCallee->getZExtValue(), getCurSDLoc(),
+                                /*isTarget=*/true);
----------------
arsenm wrote:
> This loses using the right address space sized pointer
Should I just assign ActualCallee with ConstCallee?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67147





More information about the llvm-commits mailing list