Wizard accepted this revision.
Wizard added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D43581#1016318, @stephanemoore wrote:
> In https://reviews.llvm.org/D43581#1016300, @Wizard wrote:
> > Please update the warning info to indicate that prefix 'k' is not the only
> > option for constants. Something like:
> > "const global variable '%0' must have an appropriate prefix or a name
> > which starts with 'k[A-Z]'"
> Is the latest warning info satisfactory? I believe I updated it after adding
> you as a reviewer.
Hmm I feel it is a bit unfriendly to show users a rather complicated regex in
the info, but I will leave it to you. Not a big problem.
rCTE Clang Tools Extra
cfe-commits mailing list