This is an archive of the discontinued LLVM Phabricator instance.

[clang][Interp] Only generate disassembly in debug builds
ClosedPublic

Authored by tbaeder on Jan 27 2023, 12:19 AM.

Details

Summary
With the current set of opcodes, this saves 3460 lines in the generated
Opcodes.inc in release builds (-17%).

The dump() code is only used for debugging right now as far as I know (there is nothing equivalent to -ast-dump), so only generate this code for debug builds.

Diff Detail

Event Timeline

tbaeder created this revision.Jan 27 2023, 12:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 27 2023, 12:19 AM
tbaeder requested review of this revision.Jan 27 2023, 12:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 27 2023, 12:19 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
erichkeane accepted this revision.Feb 3 2023, 6:38 AM
This revision is now accepted and ready to land.Feb 3 2023, 6:38 AM
This revision was landed with ongoing or failed builds.Feb 3 2023, 6:40 AM
This revision was automatically updated to reflect the committed changes.