Split 'qualifier on reference type has no effect' out into a new flag
authorLuna Kirkby <llvm@moonbase.lgbt>
Fri, 6 Aug 2021 11:08:17 +0000 (07:08 -0400)
committerAaron Ballman <aaron@aaronballman.com>
Fri, 6 Aug 2021 11:09:16 +0000 (07:09 -0400)
commit6385abd0c4490e0516cb31c0b86c0fbcc052f815
tree447092fe26ac791e8a7d00a66000ed99d8d60f76
parent5173854f1994a8cbeb1da0ca074e8f9bcff2be92
Split 'qualifier on reference type has no effect' out into a new flag

This introduces a new flag ignored-reference-qualifiers for the
existing "'A' qualifier on reference type B has no effect" diagnostic,
as a child of ignored-qualifiers.

Rationale:
This particular diagnostic is enabled by default, but other parts of
ignored-qualifiers are not. Anecdotally, a user may encounter this
diagnostic in the wild, and, seeing it to be valuable, might try to
raise it to error with -Werror=ignored-qualifiers, whereupon the other
diagnostics the flag covers will also be raised, to the user's surprise
and confusion. By splitting this diagnostic out into a separate flag,
and marking it as a child of ignored-qualifiers, we allow the user more
granular control of the diagnostics they care about, while maintaining
backwards compatibility with existing build scripts.
clang/include/clang/Basic/DiagnosticGroups.td
clang/include/clang/Basic/DiagnosticSemaKinds.td