[PATCH] D110493: [clang-tidy] Fix bug in readability-uppercase-literal-suffix

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 27 11:28:02 PDT 2021


carlosgalvezp added a comment.

Thanks for the clarification, makes sense! I got the notification about the build failure, thanks for fixing. I actually added it in the first place but then noticed that checkers typically didn't include any STL header so I thought maybe there was a restriction on their usage or something. The code compiled just fine when running the "clang-check-tools" target. Good to know for the next time :)

PS: I've also reported the build errors in pre-merge, maybe they would have caught it earlier.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D110493/new/

https://reviews.llvm.org/D110493



More information about the cfe-commits mailing list