[PATCH] D44338: [LV][VPlan] Build plain CFG with simple VPInstructions for outer loops.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 9 22:32:01 PDT 2018


fhahn added a comment.

In https://reviews.llvm.org/D44338#1093996, @dcaballe wrote:

> Thanks, Florian! Some comments below.
>
> > One thing worth discussing briefly before this goes in may be what the plan for dealing with debug info will be with VPlan. Adding @aprantl in case he has some thoughts.
>
> I'm not aware of any particular proposal for debug info in VPlan at this point but I will check with my team. Currently, DbgInfoIntrinsic would be represented as a regular VPInstruction. We could think about if a specific representation for this is necessary in VPlan.


Great thanks. Besides the DbgInfoIntrinsics,  do we need some way to attach the debug metadata from the original instructions to the VPInstructions? I suppose initially we could get them from the underlying values, but IIUC some VPlan transformations could introduce new VPInstructions without underlying values.


https://reviews.llvm.org/D44338





More information about the llvm-commits mailing list