[llvm] [NVPTX] Load/Store/Fence syncscope support (PR #106101)
Artem Belevich via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 11 15:47:01 PDT 2024
================
@@ -118,6 +119,8 @@ class NVPTXSubtarget : public NVPTXGenSubtargetInfo {
NVPTXSubtarget &initializeSubtargetDependencies(StringRef CPU, StringRef FS);
void ParseSubtargetFeatures(StringRef CPU, StringRef TuneCPU, StringRef FS);
+
+ void requireClusters(std::string const &FailureMessage) const;
----------------
Artem-B wrote:
We need to make clear that this produces a fatal error if conditions are not met.
`require` alone does not carry "I am a fatal error" vibe.
I think we need something painfully obvious, along the lines of `failIfClustersNotAvailable()`.
https://github.com/llvm/llvm-project/pull/106101
More information about the llvm-commits
mailing list