[PATCH] D36851: [analyzer] Fix modeling of ctors
Devin Coughlin via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 17 16:47:21 PDT 2017
dcoughlin added a comment.
Thanks for the patch!
@NoQ and I were discussing this approach this morning. 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).
More information about the cfe-commits