alexshap added a comment.
>One alternative we discussed was performing this logic in RegionStore instead and skipping the default binding there >if we saw that the base region was empty. What do you think of that approach? (We would have to be careful for exactly the reasons described in your FIXME) yeah, i thought about this option as well, i can update my diff to try that and see how it works. Repository: rL LLVM https://reviews.llvm.org/D36851 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits