[PATCH] D115123: [VPlan] Add Debugloc to VPInstruction (NFC).

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 13 05:11:13 PST 2021


fhahn updated this revision to Diff 393859.
fhahn added a comment.



In D115123#3188547 <https://reviews.llvm.org/D115123#3188547>, @Ayal wrote:

> In D115123#3188465 <https://reviews.llvm.org/D115123#3188465>, @fhahn wrote:
>
>> But I updated the patch to make it non-optional and push the responsiblity to the places where instructions are created. I think in most cases we potentially have a location available and I tried to use it.
>
> Very good! Is it possible to test?

Yes! I precommitted both a codegen test and a VPlan printing test. The changes are now included. The patch is not NCF any longer, I'll update the description accordingly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115123

Files:
  llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/lib/Transforms/Vectorize/VPlan.cpp
  llvm/lib/Transforms/Vectorize/VPlan.h
  llvm/lib/Transforms/Vectorize/VPlanPredicator.cpp
  llvm/lib/Transforms/Vectorize/VPlanSLP.cpp
  llvm/test/Transforms/LoopVectorize/if-pred-non-void.ll
  llvm/test/Transforms/LoopVectorize/vplan-printing.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D115123.393859.patch
Type: text/x-patch
Size: 10811 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211213/3408e0b8/attachment-0001.bin>


More information about the llvm-commits mailing list