While at it, fix some typos.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Thanks for the fixed in the test, this should not require assertions though, I need to investigate what does not work as I intend! :)
Comment Actions
There's this NDEBUG check https://github.com/llvm/llvm-project/blob/main/mlir/lib/Debug/DebugCounter.cpp#L113
Comment Actions
Thanks @bkramer ! Re-enabled in https://reviews.llvm.org/D149232 after removing the macro guard and confirming the test works with asserts disabled now.