<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Okay, works for me. The patch looks good as is. I just wanted to raise the point that BasicStore after all is "basic", and I'm not certain if it makes much more sense to extend it to model more of C++.<div><br><div><div>On Nov 15, 2010, at 3:55 PM, Marcin Świderski wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote"><div class="gmail_quote">I'm currently using it for tracking object identity and more precicly checking if object is the 'this' object.</div><div class="gmail_quote"><br></div><div class="gmail_quote">
W dniu 16 listopada 2010 00:49 użytkownik Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com" target="_blank">kremenek@apple.com</a>></span> napisał:<div><div></div><div class="h5"><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">I don't think BasicStore will ever be useful for analyzing C++ code. I'm fine with restricting its use to C and Objective-C code at this point. There's just so much it won't model for C++ to be remotely useful, and the maintenance isn't worth it.<br>
<div><br></div><div>Any objections?</div><div><br><div><div><div></div><div><div>On Nov 15, 2010, at 3:47 PM, Marcin Świderski wrote:</div><br></div></div><blockquote type="cite"><div><div></div><div>
Patch adds support for CXXThisRegion in BasicStore. CXXThisRegion is treated like VarRegion and ObjCIVarRegion. Also there's small change in BindDeclInternal to treat reference decls like pointer decls.<div><div><br>
</div>
<div>Please aprove for commit.</div><div><br></div><div>- Marcin</div></div>
</div></div><span><gr-eng-cxx-this-reg-in-basic-store.patch></span></blockquote></div></div></div></blockquote></div></div></div></div><br>
_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits<br></blockquote></div><br></div></body></html>