[PATCH] D12154: [x86] invert logic for attribute 'FeatureFastUAMem'

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 19 10:05:27 PDT 2015


RKSimon added inline comments.

================
Comment at: lib/Target/X86/X86.td:489
@@ -486,1 +488,3 @@
+                               FeatureFSGSBase, FeatureSlowUAMem]>;
 
+def : Proc<"geode",           [FeatureSlowUAMem, Feature3DNowA]>;
----------------
You can drop FeatureSlowUAMem for BD targets - the AMD 15h SOG confirms that unaligned performance should be the same for aligned addresses and only +1cy for unaligned. It might be more complex for cache-line crossing but most targets will suffer there, not just BD.


http://reviews.llvm.org/D12154





More information about the llvm-commits mailing list