This is an archive of the discontinued LLVM Phabricator instance.

[VPlan] Replace LLVM_ATTRIBUTE_USED with ifndef NDEBUG
ClosedPublic

Authored by dcaballe on May 29 2018, 2:56 PM.

Details

Summary

Minor replacement. LLVM_ATTRIBUTE_USED was introduced to silence a warning but using #ifndef NDEBUG makes more sense in this case.

Diff Detail

Event Timeline

dcaballe created this revision.May 29 2018, 2:56 PM
dblaikie accepted this revision.May 29 2018, 3:07 PM

Looks good - thanks!

This revision is now accepted and ready to land.May 29 2018, 3:07 PM
This revision was automatically updated to reflect the committed changes.