[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