[PATCH] Rename getMaximumUnrollFactor().

Sanjay Patel spatel at rotateright.com
Tue Aug 26 08:28:24 PDT 2014


Hi aschwaighofer, hfinkel, nadav,

"getMaximumUnrollFactor()" isn't used by the loop unroller at all. It's strictly a loop vectorizer variable, so I propose renaming it to "getMaxVectorUnrollFactor()" with this patch. No functional change intended.

Ideally, I think we would move this subtarget-specific variable into the SchedMachineModel. That's where we already have a variable for "LoopMicroOpBufferSize" which is used by the loop unroller. If that's a good change, let me know, and I'll work on a patch for it.

http://reviews.llvm.org/D5066

Files:
  include/llvm/Analysis/TargetTransformInfo.h
  lib/Analysis/TargetTransformInfo.cpp
  lib/CodeGen/BasicTargetTransformInfo.cpp
  lib/Target/AArch64/AArch64TargetTransformInfo.cpp
  lib/Target/ARM/ARMTargetTransformInfo.cpp
  lib/Target/PowerPC/PPCTargetTransformInfo.cpp
  lib/Target/R600/AMDGPUTargetTransformInfo.cpp
  lib/Target/X86/X86TargetTransformInfo.cpp
  lib/Transforms/Vectorize/LoopVectorize.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5066.12949.patch
Type: text/x-patch
Size: 6625 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140826/3fb6531f/attachment.bin>


More information about the llvm-commits mailing list