[PATCH] [SimplifyCFG] Swap to using TargetTransformInfo for cost analysis.

[James Molloy]

Hi Andrea,

Performance on AArch64 showed no major swings. However, it sounds to me
like your TargetTransformInfo isn't behaving correctly. That's TTI's entire
point, and the x86 one should surely return "TCC_Expensive" for such
intrinsic calls.

I see that the default TTI does return TCC_Basic for almost all intrinsics,
and that is arguable behaviour. But there's no reason you shouldn't
override this. In fact, it seems the right thing to do.

Cheers,

James

On Tue Feb 10 2015 at 2:53:52 PM Andrea Di Biagio <
Andrea_DiBiagio at sn.scee.net> wrote:

> REPOSITORY
>   rL LLVM
>
> http://reviews.llvm.org/D7506
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150210/a256d9fc/attachment.html>