Here's an updated version of the patch which uses StringRef as requested.<div><br></div><div>Also, I have changed the method names to avoid the ambiguity problems encountered in the last patch. The new methods end with "Ex" which can stand either for "Expression" or "Extension" depending on your preference. :)<br>

<br><div class="gmail_quote">On Sat, Oct 17, 2009 at 9:32 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im"><br>
On Oct 5, 2009, at 10:48 PM, Talin wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The attached patch adds some additional convenience methods to DebugFactory. Specifically, methods that take a uint64 for size, alignment or offset, can now also take a Constant *. This is very useful for target-independent frontends which can use things like ConstantExpr::getSizeof(). I've been using this for about a week in my own code and it seems to work fine,<br>


</blockquote>
<br></div>
Hi Talin,<br>
<br>
Sorry for the delay, still getting caught up.  Please change this to use StringRef like the current DebugInfo.h methods do.<br>
<br>
Devang, can you review this in more detail?<br><font color="#888888">
<br>
-Chris<br>
<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>-- Talin<br>
</div>