[PATCH] D30364: AArch64 : Add PreferCSEL feature for Exynos-M3.

James Molloy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 1 00:24:18 PST 2017


jmolloy added a comment.

Hi Renato,

>   If it does, but the cost is bad (for whatever reason), this really really should be in the cost model...

For my own understanding - how does the cost model impact the choices ISel makes? I didn't think it could. Or are you saying *in addition to* disabling the relevant Pat<>s with a feature, this should *also* be reflected in the cost model?

James


https://reviews.llvm.org/D30364





More information about the llvm-commits mailing list