[PATCH] D58412: [X86] Remove FeatureSlowIncDec from Sandy Bridge and later Intel Core CPUs

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 19 16:36:42 PST 2019


craig.topper added a comment.

In D58412#1403190 <https://reviews.llvm.org/D58412#1403190>, @spatel wrote:

> Should we change the generic model too then since most CPUs prefer inc/dec? 
>  I'm guessing that's a lot more test diffs, so no problem to make it another patch, but just want to know if that's the way to think about the generic model.


Related question to that. Should "pentium4" which is clang's Linux default CPU for 32-bit builds have more tuning flags on it?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58412/new/

https://reviews.llvm.org/D58412





More information about the llvm-commits mailing list