[PATCH] D66733: [analyzer] Add a checker option to detect nested dead stores
    Kristóf Umann via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Oct 14 05:19:59 PDT 2019
    
    
  
Szelethus added a comment.
In D66733#1706521 <https://reviews.llvm.org/D66733#1706521>, @sylvestre.ledru wrote:
> I added it to the release notes here : https://reviews.llvm.org/rC374593
>  I am wondering if the option( WarnForDeadNestedAssignments ) to disable it is really necessary?
>  I haven't seen any false positive while deadstore has some.
Thank you!! Well, the option was in place in case some false positives do arise from this, but we did suspect that wouldn't happen. I'm not against removing it completely, or hiding it (so that it would only be listed under `-analyzer-checker-option-help-developer`).
Some of folks working on the analyzer are preparing for the conference, hence the slower response time, sorry :^)
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