<div dir="ltr"><div>Thanks for the review.  Committed as r175616.<br><br></div>Logan<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 20, 2013 at 6:31 AM, Jim Grosbach <span dir="ltr"><<a href="mailto:grosbach@apple.com" target="_blank">grosbach@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Logan,<br>
<br>
I'm not 100% familiar with the lifetime intrinsics, but I agree with Bill that this change seems safe and totally reasonable. I have no objections.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Jim<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Feb 19, 2013, at 2:25 PM, Bill Wendling <<a href="mailto:wendling@apple.com">wendling@apple.com</a>> wrote:<br>
<br>
> Hi Logan,<br>
><br>
> This LGTM and fairly obvious. I'd like Jim or Owen to chime in though.<br>
><br>
> -bw<br>
><br>
> On Feb 19, 2013, at 8:03 AM, Logan Chien <<a href="mailto:tzuhsiang.chien@gmail.com">tzuhsiang.chien@gmail.com</a>> wrote:<br>
><br>
>> Ping?<br>
>><br>
>><br>
>> On Fri, Feb 15, 2013 at 1:45 PM, Logan Chien <<a href="mailto:tzuhsiang.chien@gmail.com">tzuhsiang.chien@gmail.com</a>> wrote:<br>
>> Hi,<br>
>><br>
>>  I found that some code with llvm.lifetime.* intrinsics will raise assertion failure in Thumb1 backend when stack coloring is enabled.  The root cause is that we are accessing the stack slot for frame pointer without checking the existence of the stack slot.  The test cases and the patch are attached.  Please have a look.  Thanks.<br>

>><br>
>> Sincerely,<br>
>> Logan<br>
>><br>
>> _______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
<br>
</div></div></blockquote></div><br></div>