[PATCH] D40913: TLI: Allow using PSV for intrinsic mem operands

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 14 12:25:57 PST 2017


hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: include/llvm/CodeGen/TargetLowering.h:723
   virtual bool getTgtMemIntrinsic(IntrinsicInfo &, const CallInst &,
+                                  MachineFunction &,
                                   unsigned /*Intrinsic*/) const {
----------------
arsenm wrote:
> hfinkel wrote:
> > Why are you adding this?
> You need the MachineFunction to create a new PseudoSourceValue. Target custom pseudo types live in the target's MachineFunctionInfo
Ah, right. That's how you get the PseudoSourceValueManager reference.


https://reviews.llvm.org/D40913





More information about the llvm-commits mailing list