NoQ added a comment.

> Because string length is for now only composed of `CStringChecker`-tagged 
> metadata symbols and constants...

Even if this was not the case, it is stil certain that string length is a 
`NonLoc`. And as such it is either a plain `nonloc::SymbolVal` that is 
unaffected or a `nonloc::LocAsInteger` that is affected but isn't going to 
work, for a reason described in the `SymbolReaper::markElementIndicesLive()` 
section. This is a much more obvious way to demonstrate that `CStringChecker` 
is not affected by this patch.


https://reviews.llvm.org/D44347



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to