[PATCH] D66733: [analyzer] Add a checker option to detect nested dead stores

Balázs Benics via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 15 11:45:31 PDT 2019


steakhal added a comment.

Thank you guys the responses. I cannot agree more. The sole reason why this option exists is that if you scroll back in the git blame of that line, you would find a commit, which removed this warning for this exact scenario.
Possibly due to some seemingly false positives.
I've added the author of that patch to the reviewers of this change, but did not respond.

If you interested to dig deeper, you could start here:
rC123394 <https://reviews.llvm.org/rC123394> same on github: https://github.com/steakhal/llvm-project/commit/f224820b45c6847b91071da8d7ade59f373b96f3

And again, thank you that mentioned it in the release notes and for the kind responses as well.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D66733





More information about the llvm-commits mailing list