[PATCH] D83824: [NFC] [LV] Vectorized Loop Skeleton Refactoring

Bardia Mahjour via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 24 12:21:39 PDT 2020


bmahjour added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:3048
+BasicBlock *InnerLoopVectorizer::completeLoopSkeleton(Loop *L,
+                                                      MDNode *OrigLoopID) {
+  assert(L && "Expected valid loop.");
----------------
pjeeva01 wrote:
> Should we assert if OrigLoopID is null?
No, because the original loop may not have any metadata to begin with.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83824





More information about the llvm-commits mailing list