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
_
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
___
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
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
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
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
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
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
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