[llvm] [AArch64] Avoid streaming mode hazards from FP constant stores (PR #114207)

Benjamin Maxwell via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 31 08:30:55 PDT 2024


================
@@ -489,6 +489,10 @@ class TargetLoweringBase {
     return true;
   }
 
+  /// Returns true if a floating-point load or store can be replaced with an
+  /// equivalent integer load or store without negatively affecting performance.
+  virtual bool canUseIntLoadStoreForFloatValues() const { return true; }
----------------
MacDue wrote:

I'm not sure... :sweat_smile: To me, "can" means you _can_ do something if it's beneficial, which using integer stores for FP constants may be. And "should" means you should _always_ do something, but it's not the case that you should always use integer loads for FP values. 

https://github.com/llvm/llvm-project/pull/114207


More information about the llvm-commits mailing list