[PATCH] D29487: Make RWPI use movw/movt when available

[Oliver Stannard via Phabricator via llvm-commits]

olista01 added a comment.

The test change removes the tests of the constant-pool code path, but it looks like this path will still be used when optimising for size and not using execute-only. Could you extend the tests to check both code-paths?



================
Comment at: lib/Target/ARM/ARMISelLowering.cpp:3195
+      SDValue RelAddr = DAG.getNode(ARMISD::Wrapper, dl, PtrVT, G);
+      SDValue SB = DAG.getCopyFromReg(DAG.getEntryNode(), dl, ARM::R9, PtrVT);
+      SDValue Result = DAG.getNode(ISD::ADD, dl, PtrVT, SB, RelAddr);
----------------
The getCopyFromReg and ISD::ADD are duplicated, could they be pulled below the if?


Repository:
  rL LLVM

https://reviews.llvm.org/D29487