[PATCH] D49488: [LV] Move InterleaveGroup and InterleavedAccessInfo to VectorUtils.h (NFC)

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 21 09:47:38 PDT 2018


fhahn added inline comments.


================
Comment at: include/llvm/Analysis/VectorUtils.h:263
+    int Key = SmallestKey + Index;
+    if (!Members.count(Key))
+      return nullptr;
----------------
xbolva00 wrote:
> Do we need count?
> 
> if (Members.find(Key) == Members.end())
>   return nullptr;
> 
> ?
No need to count here I think, but I would prefer to keep this change NFC.  Feel free to fix this in the the current version in LoopVectorize.cpp. Otherwise I can make the count() change separately too.

Before submitting the patch, I will make sure it contains any changes made the LoopVectorize.cpp's version of the code. 


https://reviews.llvm.org/D49488





More information about the llvm-commits mailing list