[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