RKSimon added inline comments.

================
Comment at: llvm/lib/Target/X86/X86.td:790
+                                                  FeatureFSRM];
+  list<SubtargetFeature> RKLTuning = ICLTuning;
+  list<SubtargetFeature> RKLFeatures =
----------------
Using ICLTuning suggests we should still be avoiding 512-bit ops 
(FeaturePrefer256Bit) - is this still true for RKL (or anything past CNL...)?

I posted PR48336 but never got any response, but from what others have reported 
(Travis Downs, Phoronix etc) its mainly a power issue these days, not a perf 
issue due to big freq drops.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100085

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to