[Mlir-commits] [llvm] [mlir] [MLIR][NVVM] Add NVVMRequiresSM op trait (PR #126886)

[Durgadoss R]

================
@@ -1793,6 +1793,22 @@ void GPUModuleOp::setTargets(ArrayRef<TargetAttrInterface> targets) {
   targetsAttr = ArrayAttr::get(getContext(), targetsVector);
 }
 
+LogicalResult GPUModuleOp::verify() {
+  auto targets = getOperation()->getAttrOfType<ArrayAttr>("targets");
+
+  if (!targets)
+    return success();
+
----------------
durga4github wrote:

yes, I was not thinking about compile-time here.

I would like to have an option to keep these checks "out-of-the-way" during certain development flows.
Especially when we build/test the lowering-upto-llvmir only and not further below.

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