This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Disable -Wconstant-evaluated for testsuite
ClosedPublic

Authored by xbolva00 on Nov 1 2019, 5:46 AM.

Diff Detail

Event Timeline

xbolva00 created this revision.Nov 1 2019, 5:46 AM
Herald added a project: Restricted Project. · View Herald Transcript
This revision was not accepted when it landed; it landed in state Needs Review.Nov 1 2019, 5:49 AM
This revision was automatically updated to reflect the committed changes.

Why was this committed without review?

Why are we ignoring this warning instead of fixing it?

xbolva00 added a comment.EditedNov 18 2019, 2:17 PM

Yes, it was commited to fix red buildbot (libc++’s buildbot uses -Werror).

Warning did not reveal any real bugs in testsuite. It warned for cases which are tautologically true/false but it makes sense to have such tests in libc++ testsuite. So turning off warning looked as a reasonable decision.