[PATCH] D74486: [PowerPC][Future] Add initial support for PC Relative addressing for constant pool loads

Amy Kwan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 1 14:01:14 PST 2020


amyk added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:2541
 
+bool PPCTargetLowering::SelectAddressPCRel(SDValue N, SDValue &Base) const {
+  ConstantPoolSDNode *ConstPoolNode =
----------------
I think it would be good if we had doxygen /// comments for this. 


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrPrefix.td:375
+  // If the MATpcreladdr node is not caught by any other pattern it should be
+  // caught here and turned into a paddi instruction to meterialize the address.
+  def : Pat<(MATpcreladdr pcreladdr:$cp), (PADDI8pc 0, $cp)>;
----------------
s/meterialize/materialize 


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

https://reviews.llvm.org/D74486





More information about the llvm-commits mailing list