This is an archive of the discontinued LLVM Phabricator instance.

[Bazel] Don't fail the build on usage of deprecated APIs
ClosedPublic

Authored by GMNGeoffrey on Jan 31 2022, 5:54 PM.

Details

Summary

Build failures are not a particularly helpful way to enforce not using
deprecated APIs and that isn't the point of the Bazel build.

At the same time, this removes -Wno-unused this is a check that we do
enforce in the Google internal build and so are ok maintaining in our
maintenance of the upstream Bazel build (the comment about not wanting
to do so was from a time when this was in a separate repository and I was
the only one maintaining it).

Diff Detail

Event Timeline

GMNGeoffrey requested review of this revision.Jan 31 2022, 5:54 PM
GMNGeoffrey created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJan 31 2022, 5:54 PM
This revision was not accepted when it landed; it landed in state Needs Review.Jan 31 2022, 6:10 PM
This revision was automatically updated to reflect the committed changes.