[llvm] [AArch64] Add getStreamingHazardSize() to AArch64Subtarget (PR #113679)

David Green via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 25 07:47:44 PDT 2024


================
@@ -472,7 +482,8 @@ AArch64TargetMachine::getSubtargetImpl(const Function &F) const {
     resetTargetOptions(F);
     I = std::make_unique<AArch64Subtarget>(
         TargetTriple, CPU, TuneCPU, FS, *this, isLittle, MinSVEVectorSize,
-        MaxSVEVectorSize, IsStreaming, IsStreamingCompatible, HasMinSize);
+        MaxSVEVectorSize, IsStreaming, IsStreamingCompatible,
+        StreamingHazardSize, HasMinSize);
----------------
davemgreen wrote:

I never know this layering very well, but is there a strong reason to go via AArch64TargetMachine, as opposed to just setting the variable in AArch64Subtarget::AArch64Subtarget? I would expect this might become a target-feature in the end, which could be overridden by the option. That might be simpler if it was contained within AArch64SubTarget.

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


More information about the llvm-commits mailing list