[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