<div dir="ltr">I feel confident enough acking this one if it helps as well.<div><br></div><div>-eric<br><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 30, 2015 at 10:27 AM Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Since Evan seems to be unavailable, and this has been in trunk for<br>
almost a week without issues, I'll go ahead and merge it.<br>
<br>
r243662.<br>
<br>
 - Hans<br>
<br>
<br>
On Wed, Jul 29, 2015 at 10:16 AM, Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
> Ping?<br>
><br>
> On Mon, Jul 27, 2015 at 1:25 PM, Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
>> Ping?<br>
>><br>
>> On Fri, Jul 24, 2015 at 9:28 AM, Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<br>
>>> Seems reasonable.<br>
>>><br>
>>> Evan, I believe you're the code owner here. OK to merge to 3.7?<br>
>>><br>
>>> On Fri, Jul 24, 2015 at 8:57 AM, Robinson, Paul<br>
>>> <<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.sony.com</a>> wrote:<br>
>>>> Can we please have this merged to the 3.7 branch?<br>
>>>> Thanks,<br>
>>>> --paulr<br>
>>>><br>
>>>>> -----Original Message-----<br>
>>>>> From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu" target="_blank">llvm-commits-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:llvm-commits-" target="_blank">llvm-commits-</a><br>
>>>>> <a href="mailto:bounces@cs.uiuc.edu" target="_blank">bounces@cs.uiuc.edu</a>] On Behalf Of Sanjay Patel<br>
>>>>> Sent: Thursday, July 23, 2015 3:57 PM<br>
>>>>> To: <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
>>>>> Subject: [llvm] r243057 - fix crash in machine trace metrics due to<br>
>>>>> processing dbg_value instructions (PR24199)<br>
>>>>><br>
>>>>> Author: spatel<br>
>>>>> Date: Thu Jul 23 17:56:53 2015<br>
>>>>> New Revision: 243057<br>
>>>>><br>
>>>>> URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D243057-26view-3Drev&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=8PCeVgG8LaNb_T07PIJWx9t6sg_c4YrN_J1dZQbsrbs&s=Pyo9wFL2h5rMJ3IyHnvMQ6-AQIt4uo4nah0dl4ZbGmg&e=" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=243057&view=rev</a><br>
>>>>> Log:<br>
>>>>> fix crash in machine trace metrics due to processing dbg_value<br>
>>>>> instructions (PR24199)<br>
>>>>><br>
>>>>> The test in PR24199 ( <a href="https://urldefense.proofpoint.com/v2/url?u=https-3A__llvm.org_bugs_show-5Fbug.cgi-3Fid-3D24199&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=8PCeVgG8LaNb_T07PIJWx9t6sg_c4YrN_J1dZQbsrbs&s=CJp02QMTTF_mmMXYykgBsvDc7ljy92DJY4Apib-c5jM&e=" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=24199</a> )<br>
>>>>> crashes because machine<br>
>>>>> trace metrics was not ignoring dbg_value instructions when calculating<br>
>>>>> data dependencies.<br>
>>>>><br>
>>>>> The machine-combiner pass asks machine trace metrics to calculate an<br>
>>>>> instruction trace,<br>
>>>>> does some reassociations, and calls<br>
>>>>> MachineInstr::eraseFromParentAndMarkDBGValuesForRemoval()<br>
>>>>> along with MachineTraceMetrics::invalidate(). The dbg_value instructions<br>
>>>>> have their operands<br>
>>>>> invalidated, but the instructions are not expected to be deleted.<br>
>>>>><br>
>>>>> On a subsequent loop iteration of the machine-combiner pass, machine trace<br>
>>>>> metrics would be<br>
>>>>> called again and die while accessing the invalid debug instructions.<br>
>>>>><br>
>>>>> Differential Revision: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11423&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=8PCeVgG8LaNb_T07PIJWx9t6sg_c4YrN_J1dZQbsrbs&s=ImS6UPRbHAn9OHsdx7KrRUd4LF6mvSlXSVsl5IzK-LY&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11423</a><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div></div>