martong marked 2 inline comments as done.
martong added a comment.

> Can we enable the check by default then? What pieces are missing? Like, the 
> symbolic extent/offset case is not a must. I think we have everything except 
> maybe some deeper testing. I'd rather spend some time on the above exercise, 
> but then enable by default if it goes well.

I am going to execute the analysis on LLVM/Clang as you suggested, so then 
we'll see what the experiment will bring us. And then we could enable it by 
default I think. BTW how can we do that? I could not find any default 
enablement related config in Checkers.td ...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71612



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

Reply via email to