This is an archive of the discontinued LLVM Phabricator instance.

[mlir] ActionTracing: require ASSERTS for debugcounter test
ClosedPublic

Authored by cota on Apr 25 2023, 7:27 AM.

Details

Diff Detail

Event Timeline

cota created this revision.Apr 25 2023, 7:27 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 25 2023, 7:27 AM
cota requested review of this revision.Apr 25 2023, 7:27 AM
bkramer accepted this revision.Apr 25 2023, 7:30 AM
This revision is now accepted and ready to land.Apr 25 2023, 7:30 AM
This revision was landed with ongoing or failed builds.Apr 25 2023, 7:59 AM
This revision was automatically updated to reflect the committed changes.

Thanks for the fixed in the test, this should not require assertions though, I need to investigate what does not work as I intend! :)

Thanks for the fixed in the test, this should not require assertions though, I need to investigate what does not work as I intend! :)

There's this NDEBUG check https://github.com/llvm/llvm-project/blob/main/mlir/lib/Debug/DebugCounter.cpp#L113

Thanks @bkramer ! Re-enabled in https://reviews.llvm.org/D149232 after removing the macro guard and confirming the test works with asserts disabled now.