[Mlir-commits] [mlir] [MLIR][NVVM] Add `dumpISA` and `dumpMachineISA` Flags (PR #116199)

Mehdi Amini llvmlistbot at llvm.org
Thu Nov 14 03:21:40 PST 2024


https://github.com/joker-eph requested changes to this pull request.

> Currently, dumping virtual and machine-level ISA is restricted to debug builds, making it unavailable in release builds. However, the ability to view these ISAs can be valuable even in release builds or with production compilers.

I don't buy this argument: debug output remains debug output. You're just arbitrarily interested in some debug output instead of others, but that does not seem like well plumbed to me here.

See https://github.com/llvm/llvm-project/pull/116007 for a more principled approach that offers the actually features in a more structured way.

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


More information about the Mlir-commits mailing list