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

[Mehdi Amini]

joker-eph wrote:

Can you add the motivation and usage in the description of the PR? Right now this describe what it does but not really provide context on why.

> The check can be disabled with the existing --mlir-very-unsafe-disable-verifier-on-parsing flag (tests added in nvvm-check-targetSM-disabled.mlir)

Please remove this: 1) as the name indicates, this isn't something we'd want anyone to use, and 2) this is misleading: it does not disable any check other than on initial parsing.

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