[llvm] [AArch64] NFC: Rename -force-streaming-compatible-sve to -force-streaming-compatible (PR #92774)
Paul Walker via llvm-commits
llvm-commits at lists.llvm.org
Mon May 20 09:17:42 PDT 2024
================
@@ -548,7 +546,7 @@ void AArch64Subtarget::mirFileLoaded(MachineFunction &MF) const {
bool AArch64Subtarget::useAA() const { return UseAA; }
bool AArch64Subtarget::isStreamingCompatible() const {
- return StreamingCompatibleSVEMode || ForceStreamingCompatibleSVE;
+ return IsStreamingCompatible || ForceStreamingCompatible;
----------------
paulwalker-arm wrote:
Given you're cleaning this up, please move the `ForceStreamingCompatible` abstraction into `AArch64TargetMachine::AArch64TargetMachine`. You'll see we ended up doing the same for the sve-vector-bits handling so the command line option controls the creation of subtarget rather than needing to overriding its state like done here.
Although this'll make the patch non-NFC, I argue that's already the case given the PR is already removing one command line option to introduce a new one.
https://github.com/llvm/llvm-project/pull/92774
More information about the llvm-commits
mailing list