<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Feb 25, 2014 at 7:11 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Hi Adrian,<br>
<br>
  This is a lot of great work, thanks!<br>
<br>
  I think the first question is how this is going to interact with DIBuilder::createComplexVariable? I think it should just be a concatenation of the two things. It looks like you have it as an instead?</blockquote><div>
<br></div><div>Missed a word here? ("It looks like you have it as an <thing> instead?")</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 In general, I think I like the idea that variable locations just have expressions attached to them. And lacking anything else I think dwarf expressions are just fine here.<br>
<br>
  I've got more comments on the patch itself, but let's get this out of the way first.<br>
<br>
  -eric<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2680" target="_blank">http://llvm-reviews.chandlerc.com/D2680</a><br>
</blockquote></div><br></div></div>