[PATCH] D157429: [NFC] [Clang] Fix static analyzer concern

2023-08-08 Thread Elizabeth Andrews via Phabricator via cfe-commits
eandrews added a comment. In D157429#4570540 , @aaron.ballman wrote: > LGTM with a small formatting nit. Thanks for the review! I committed the patch after fixing the formatting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D157429: [NFC] [Clang] Fix static analyzer concern

2023-08-08 Thread Elizabeth Andrews via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6a4779cc235c: [NFC] Fix static analyzer concern (authored by eandrews). Herald added a project: clang. Changed prior to commit:

[PATCH] D157429: [NFC] [Clang] Fix static analyzer concern

2023-08-08 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM with a small formatting nit. Comment at: clang/lib/Analysis/BodyFarm.cpp:809 Prop = MD->findPropertyDecl(); -IVar = findBackingIvar(Prop); +

[PATCH] D157429: [NFC] [Clang] Fix static analyzer concern

2023-08-08 Thread Elizabeth Andrews via Phabricator via cfe-commits
eandrews created this revision. eandrews added reviewers: aaron.ballman, tahonermann. Herald added subscribers: manas, ASDenysPetrov, dkrupp, donat.nagy, Szelethus, a.sidorin, baloghadamsoftware. Herald added a reviewer: NoQ. Herald added a project: All. eandrews requested review of this