[Mlir-commits] [mlir] [MLIR][NVVM] Add LLVM IR import support for nvvm.barrier (PR #188630)

[Oleksandr Alex Zinenko]

================
@@ -30,19 +31,57 @@ static bool isConvertibleIntrinsic(llvm::Intrinsic::ID id) {
 /// Returns the list of LLVM IR intrinsic identifiers that are convertible to
 /// MLIR NVVM dialect intrinsics.
 static ArrayRef<unsigned> getSupportedIntrinsicsImpl() {
-  static const SmallVector<unsigned> convertibleIntrinsics = {
+  static const SmallVector<unsigned> convertibleIntrinsics = [] {
+    SmallVector<unsigned> ids = {
 #include "mlir/Dialect/LLVMIR/NVVMConvertibleLLVMIRIntrinsics.inc"
-  };
+    };
+
+    ids.push_back(llvm::Intrinsic::nvvm_barrier_cta_sync_aligned_all);
----------------
ftynse wrote:

Won't something like

```
  static const SmallVector<unsigned> convertibleIntrinsics = {
    llvm::Intrinsic::nvvm_barrier_cta_sync_aligned_all,

#include "mlir/Dialect/LLVMIR/NVVMConvertibleLLVMIRIntrinsics.inc"
  };
```

work instead?

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