[llvm] [AArch64] Set the default streaming hazard size to 1024 for +sme,+sve (PR #123753)

Sander de Smalen via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 21 09:18:41 PST 2025


================
@@ -179,7 +179,14 @@ class AArch64Subtarget final : public AArch64GenSubtargetInfo {
 
   /// Returns the size of memory region that if accessed by both the CPU and
   /// the SME unit could result in a hazard. 0 = disabled.
-  unsigned getStreamingHazardSize() const { return StreamingHazardSize; }
+  unsigned getStreamingHazardSize() const {
+    // If StreamingHazardSize has been explicitly set to a value, use that.
+    // Otherwise, default to 1024 bytes when both SME and SVE are available
+    // (without FA64), for all other configurations default to no streaming
+    // hazards.
----------------
sdesmalen-arm wrote:

nit: this comment is redundant, because the expression below describes exactly that.

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


More information about the llvm-commits mailing list