This is an archive of the discontinued LLVM Phabricator instance.

Inline variable only used in assertion
ClosedPublic

Authored by scw on Oct 25 2019, 3:00 PM.

Details

Summary

This prevents unused variable warning/error in -DNDEBUG builds. The variable was introduced in rG5934cd11ea3e.

Diff Detail

Event Timeline

scw created this revision.Oct 25 2019, 3:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 25 2019, 3:00 PM
JDevlieghere accepted this revision.EditedOct 25 2019, 3:35 PM

LGTM.

PS: In the future please upload full-context diffs.

This revision is now accepted and ready to land.Oct 25 2019, 3:35 PM
scw updated this revision to Diff 226512.Oct 25 2019, 3:37 PM

Thanks! Please let me know if you need me to commit this for you.

scw added a comment.EditedOct 25 2019, 3:45 PM

Missed my alias and used the command line for other projects instead :/

Yes, please commit it for me as I don't have the permission. Thank you!

This revision was automatically updated to reflect the committed changes.