[PATCH] reenable gep merging in some constrainted cases

Wei Mi wmi at google.com
Thu May 21 17:49:34 PDT 2015


Hi Quentin,

I put the disabling GEP merging and related heuristic tuning under an internal option and make it the default for now. By the way, I think it is better that LoopVectorizationLegality::isConsecutivePtr can check more than one level gep, just like what DecomposeGEPExpression does now in Basicaa.  I will try that in another patch.

Thanks,
Wei.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9865

Files:
  lib/Transforms/InstCombine/InstructionCombining.cpp
  test/Transforms/InstCombine/gep-merge1.ll
  test/Transforms/InstCombine/gep-merge2.ll
  test/Transforms/InstCombine/getelementptr.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9865.26295.patch
Type: text/x-patch
Size: 13082 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150522/e5a8e800/attachment.bin>


More information about the llvm-commits mailing list