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. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D43581 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
