[PATCH] D55305: [RISCV] Add lowering of global TLS addresses

[Lewis Revill via Phabricator via llvm-commits]

lewis-revill added inline comments.
Herald added subscribers: benna, psnobl.
Herald added a project: LLVM.


================
Comment at: lib/Target/RISCV/RISCVISelLowering.cpp:468
+      0);
+  return SDValue(DAG.getMachineNode(RISCV::ADDI, DL, Ty, MNAdd, AddrLo), 0);
+}
----------------
Am I correct in thinking this sequence should actually be opaque here, it causes a DAG with multiple load/stores of the same GlobalTLSAddress to only duplicate the `ADDI` relative to a single `AddTPRel` node... I know this would be safe for absolute addressing, but not for thread-pointer relative addressing?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55305