[PATCH] D52883: [LoopVectorizer] use TTI.getOperandInfo()

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 5 07:14:59 PDT 2018


jonpa updated this revision to Diff 168468.
jonpa added a comment.

OK, I removed the initializer per your suggestion. I also removed the declarations of Op1VK and Op1VP, since this made the code more compact and also a bit more readable in the sense of which operand is actually being analyzed.

Does this look good to you?


https://reviews.llvm.org/D52883

Files:
  include/llvm/Analysis/TargetTransformInfo.h
  lib/Analysis/TargetTransformInfo.cpp
  lib/Transforms/Vectorize/LoopVectorize.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52883.168468.patch
Type: text/x-patch
Size: 7203 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181005/cd458491/attachment.bin>


More information about the llvm-commits mailing list