[PATCH] [SimplifyCFG] Swap to using TargetTransformInfo for cost analysis.
James Molloy
james.molloy at arm.com
Tue Feb 10 01:04:19 PST 2015
Hi Kristof,
This revision doesn't have any deliberate heuristic changes, so clamp() will not be optimized in this revision to min/max. I've added such a test to http://reviews.llvm.org/D7507, which is where I expect this to happen.
The AArch64 backend does not, at the moment, produce fmin/fmax for this, it produces fcsel. That is an extra improvement we need to make.
Cheers,
James
REPOSITORY
rL LLVM
http://reviews.llvm.org/D7506
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list