[PATCH] D39080: [libcxx] [test] Tolerate even more [[nodiscard]] in the STL.
Eric Fiselier via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 6 18:29:00 PST 2017
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
I think I would prefer if this patch "left an explicit trace" as to the reasons for all the `(void)` casts. Specifically I think a macro like `TEST_IGNORE_DISCARD <expr>` might be nice.
@BillyONeal What do you think?
Otherwise this LGTM.
https://reviews.llvm.org/D39080
More information about the cfe-commits
mailing list