xazax.hun added a comment.

If we end up going with this approach, I wonder if it would be a great time to 
update some of the docs here: 
https://clang.llvm.org/docs/analyzer/developer-docs/RegionStore.html
Usually, we are not doing a great job keeping these documentations up to date. 
I think the logic to determine which symbols and regions are live and how that 
logic interacts with the different types of memory regions might be important 
enough to have some documentation on it.

(Also, I think we should add link to the SA talks at the LLVM Dev Conf., but 
that is completely unrelated to this change.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134947

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

Reply via email to