[Mlir-commits] [mlir] [mlir][ArmSME] Make use of backend function attributes for enabling ZA storage (PR #71044)

Andrzej WarzyƄski llvmlistbot at llvm.org
Tue Nov 7 03:24:21 PST 2023


================
@@ -24,15 +24,19 @@ namespace arm_sme {
 // the function interface (ABI) and the caller manages PSTATE.SM on entry/exit.
 // In a locally streaming function PSTATE.SM is kept internal and the callee
 // manages it on entry/exit.
-enum class ArmStreaming { Default = 0, Locally = 1 };
+enum class ArmStreamingMode { Default = 0, Locally = 1 };
----------------
banach-space wrote:

Wondering whether this shouldn't be something like:
```cpp
enum class ArmStreamingMode { Disabled = 0, ArmStreaming, ArmLocallyStreaming = 1 };
```
Basically, we'd want this to be consistent with ACLE (which doesn't have `Default`) as well as `ArmZaMode` (which defines `Disabled`).

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


More information about the Mlir-commits mailing list