<br><br><div class="gmail_quote">On Sat, Nov 15, 2008 at 12:37 AM, 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 class="Ih2E3d"><br>
On Nov 14, 2008, at 3:54 AM, Zhongxing Xu wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
+void AllocaRegion::ProfileRegion(llvm::FoldingSetNodeID& ID,<br>
+                                 const Expr* Ex, unsigned cnt) {<br>
+  ID.AddInteger((unsigned) AllocaRegionKind);<br>
+  ID.AddPointer(Ex);<br>
+  ID.AddInteger(cnt);<br>
+}<br>
<br>
Do we need to include the superRegion in the profile?<br>
</blockquote>
<br>
<br></div>
The first value in the profile distinguishes all AllocaRegions with other MemRegions in the FoldingSet, so the last two elements in the profile are only used to distinguish between different AllocaRegion.  Currently all AllocaRegions have the same superRegion (the region for the stack) so it would provide no benefit to include it.   We of course can change this assumption if we think different AllocaRegions might have different super regions.<br>

</blockquote></div><br>Reasonable.<br>