[PATCH] D145047: Fix broken link on Clang documentation page

2023-03-02 Thread Breno Rodrigues Guimaraes via Phabricator via cfe-commits
brenoguim accepted this revision. brenoguim added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145047/new/ https://reviews.llvm.org/D145047 _

[PATCH] D31130: B32239 clang-tidy should not warn about array to pointer decay on system macros

2020-03-24 Thread Breno Rodrigues Guimaraes via Phabricator via cfe-commits
brenoguim added a comment. Oh, and if I remember correctly, the reason for this patch was to avoid exactly the github issue mentioned here. I'm embarrassed I didn't explain that anywhere :( CHANGES SINCE LAST ACTION https://reviews.llvm.org/D31130/new/ https://reviews.llvm.org/D31130 ___

[PATCH] D31130: B32239 clang-tidy should not warn about array to pointer decay on system macros

2020-03-24 Thread Breno Rodrigues Guimaraes via Phabricator via cfe-commits
brenoguim added a comment. Hey, I'm back. I actually kind of forgot about this. Unfortunately I cannot to continue the patch soon. How do I grant you full rights on this patch? If just saying here is enough, there you go, anyone can take it and change it. If there is more bureaucracy to it, jus

[PATCH] D31130: B32239 clang-tidy should not warn about array to pointer decay on system macros

2017-05-01 Thread Breno Rodrigues Guimaraes via Phabricator via cfe-commits
brenoguim updated this revision to Diff 97385. brenoguim marked an inline comment as done. brenoguim added a comment. Never flag predefinedExpr() types. https://reviews.llvm.org/D31130 Files: clang-tidy/cppcoreguidelines/ProBoundsArrayToPointerDecayCheck.cpp test/clang-tidy/Inputs/cppcoreg

[PATCH] D31130: B32239 clang-tidy should not warn about array to pointer decay on system macros

2017-03-25 Thread Breno Rodrigues Guimaraes via Phabricator via cfe-commits
brenoguim updated this revision to Diff 93062. brenoguim added a comment. - Removed the "#include " which caused problems in environments without that header - Included a directory with -isystem to simulate system headers - Added a "macro.h" header with definitions of types, functions and macros

[PATCH] D31130: B32239 clang-tidy should not warn about array to pointer decay on system macros

2017-03-21 Thread Breno Rodrigues Guimaraes via Phabricator via cfe-commits
brenoguim added a comment. I appreciate it! https://reviews.llvm.org/D31130 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D31130: B32239 clang-tidy should not warn about array to pointer decay on system macros

2017-03-21 Thread Breno Rodrigues Guimaraes via Phabricator via cfe-commits
brenoguim marked 4 inline comments as done. brenoguim added a comment. Hi @aaron.ballman, Thanks for the review! I don't have rights to push this. Who should I contact to push the change in my behalf? https://reviews.llvm.org/D31130 ___ cfe-commi

[PATCH] D31130: B32239 clang-tidy should not warn about array to pointer decay on system macros

2017-03-20 Thread Breno Rodrigues Guimaraes via Phabricator via cfe-commits
brenoguim updated this revision to Diff 92419. brenoguim added a comment. - Using the ImplicitCastExpr type on the AST_MATCHER directly to avoid explicit cast. - Using isa<> instead of dyn_cast<> to just check for type - Move variable declaration into "if" condition https://reviews.llvm.org/D3

[PATCH] D31130: B32239 clang-tidy should not warn about array to pointer decay on system macros

2017-03-19 Thread Breno Rodrigues Guimaraes via Phabricator via cfe-commits
brenoguim added a reviewer: cfe-commits. brenoguim added a comment. Forgot to add cfe-commits... https://reviews.llvm.org/D31130 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits