[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