[PATCH] D62911: WIP: AMDGPU: Use fixup for local linkage functions

[Michael Liao via Phabricator via llvm-commits]

hliao added inline comments.
Herald added a subscriber: kerbowa.


================
Comment at: lib/Target/AMDGPU/AMDGPUMCInstLower.cpp:156
+    if (MO.getTargetFlags() == SIInstrInfo::MO_PCREL32_HI)
+      Expr = AMDGPUMCExpr::create(AMDGPUMCExpr::VK_AMDGPU_PCREL_HI32, Expr, Ctx);
+
----------------
Why not use `MCBinaryExpr::createAShr` to shift that high bits into low bits directly? We don't need invent a new target fixup.


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

https://reviews.llvm.org/D62911