[PATCH] D11901: [TTI] Add a hook for specifying per-target defaults for Interleaved Accesses

silviu.baranga@arm.com via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 10 06:07:15 PDT 2015


sbaranga added inline comments.

================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:3886
@@ +3885,3 @@
+  // If an override option has been passed in for interleaved accesses, use it.
+  if (EnableInterleavedMemAccesses.getNumOccurrences() > 0)
+    UseInterleaved = EnableInterleavedMemAccesses;
----------------
rengolin wrote:
> Why the use of getNumOccurrences?
We want to use EnableInterleavedMemAccess only when the user has specified this option on the command line, otherwise we want to use the target defaults.

As far as I can tell, we need to use getNumOccurences to do that.


http://reviews.llvm.org/D11901





More information about the llvm-commits mailing list