[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