This is an archive of the discontinued LLVM Phabricator instance.

[NFC] [Clang] Fix static analyzer concern
ClosedPublic

Authored by eandrews on Aug 8 2023, 12:30 PM.

Details

Summary

Fix static analyzer concern about potential null value dereference. findBackingIvar() dereferences Prop. PR checks that Prop exists before calling the function.

Diff Detail

Event Timeline

eandrews created this revision.Aug 8 2023, 12:30 PM
Herald added a project: Restricted Project. · View Herald Transcript
eandrews requested review of this revision.Aug 8 2023, 12:30 PM
aaron.ballman accepted this revision.Aug 8 2023, 1:05 PM

LGTM with a small formatting nit.

clang/lib/Analysis/BodyFarm.cpp
809

Formatting.

This revision is now accepted and ready to land.Aug 8 2023, 1:05 PM
This revision was landed with ongoing or failed builds.Aug 8 2023, 1:28 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 8 2023, 1:28 PM

LGTM with a small formatting nit.

Thanks for the review! I committed the patch after fixing the formatting