[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