[PATCH] D114321: [LV] Move code from widenGEP to VPWidenGEPRecipe (NFC).

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 28 10:53:19 PST 2021


fhahn marked an inline comment as done.
fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:563
 
+  /// Similar to the previous function but it adds the metadata to a
+  /// vector of instructions.
----------------
Ayal wrote:
> Better keep both addMetadata() methods next to each other; the one left below is not more "protected" than this one, plus their documentations are linked - above refers to the "previous function" below.
Thanks, I moved both in the committed version.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114321



More information about the llvm-commits mailing list