[PATCH] D38406: [dump] Remove NDEBUG from test to enable dump methods [NFC]

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 6 16:11:27 PST 2017


MatzeB added a comment.

Though admittedly I like the solution here for its simplicity. However we should change the default to `ON`. The only downside is obviously external packagers need to be aware of the flag now to take advantage in their releases but I personally can live with that given that this is only about a few kilobytes saved in release libraries I believe.


https://reviews.llvm.org/D38406





More information about the llvm-commits mailing list