<br><br><div class="gmail_quote">On Thu, Nov 13, 2008 at 10:57 PM, Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com">kremenek@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div style=""><br><div><div class="Ih2E3d"><div>On Nov 12, 2008, at 11:58 PM, Zhongxing Xu wrote:</div><br></div><blockquote type="cite"><span style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px;"><div class="Ih2E3d">
Author: zhongxingxu<br>Date: Thu Nov 13 01:58:20 2008<br>New Revision: 59232<br><br>URL:<span> </span><a href="http://llvm.org/viewvc/llvm-project?rev=59232&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=59232&view=rev</a><br>
Log:<br>Lift the pointer to alloca'ed region to the pointer to its first element.<br>This is required by some operations, e.g., *p = 1; p[0] = 1;.<br>Also set the AllocaRegion's type during the cast.<br><br></div>
<div class="Ih2E3d">+        // Set the AllocaRegion's type.<br>+        const_cast<AllocaRegion*>(AR)->setType(T);</div></span></blockquote></div><br><div>Hi Zhongxing,</div><div><br></div><div>I don't think this is the way to go.  MemRegions must be immutable.  Moreover, this reasoning should not go in GRExprEngine.</div>
<div></div></div></blockquote><div><br>Hi Ted,<br><br>I rewrite the code with an AnonTypedRegion created to attach type information to. Please review it. <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div style=""><div> <br></div></div></blockquote></div><br>