<div dir="ltr">Happy to punt this one to someone who might be more familiar with the variable tracking stuff - I've not really looked at it very hard (yeah, this patch looks pretty obvious, but that just goes to show how little confidence I have in my understanding of these things)</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 4, 2014 at 3:39 PM, Robinson, Paul <span dir="ltr"><<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.sony.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for correcting this oversight!  LGTM but dblaikie gets the final<br>
word, I think.<br>
--paulr<br>
<div class="HOEnZb"><div class="h5"><br>
> -----Original Message-----<br>
> From: Amjad Aboud [mailto:<a href="mailto:amjad.aboud@intel.com">amjad.aboud@intel.com</a>]<br>
> Sent: Thursday, December 04, 2014 12:05 PM<br>
> To: <a href="mailto:amjad.aboud@intel.com">amjad.aboud@intel.com</a>; <a href="mailto:eugenis@google.com">eugenis@google.com</a>; Robinson, Paul<br>
> Cc: <a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>; <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> Subject: Re: [PATCH] Supported llvm.dbg.declare intrinsic in inline pass<br>
><br>
> <a href="http://reviews.llvm.org/D6525" target="_blank">http://reviews.llvm.org/D6525</a><br>
><br>
> Files:<br>
>   lib/Transforms/Utils/InlineFunction.cpp<br>
>   test/Transforms/Inline/inline_dbg_declare.ll<br>
</div></div></blockquote></div><br></div>