[PATCH] D90558: [VPlan] Add VPDef class.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 15 03:54:09 PST 2020


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

Fix comment, expand VPDef doc-comment to state VPDefs own their defined VPValues and are responsible for deleting them. Also mention inheritance order when inheriting from both VPValue and VPDef.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90558

Files:
  llvm/docs/Proposals/VectorizationPlan.rst
  llvm/lib/Transforms/Vectorize/VPlan.cpp
  llvm/lib/Transforms/Vectorize/VPlanValue.h
  llvm/unittests/Transforms/Vectorize/VPlanTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D90558.305349.patch
Type: text/x-patch
Size: 8281 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201115/5145d0b7/attachment-0001.bin>


More information about the llvm-commits mailing list