
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>

</head>

<body dir="ltr">

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

Cool, done<span style="font-family: calibri, arial, helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0);"> in </span><span style="color: rgb(116, 119, 125); font-family: "Segoe UI", "Segoe UI Emoji", "Segoe UI Symbol", Lato, "Helvetica Neue", Helvetica, Arial, sans-serif; font-size: 13px; text-align: left; background-color: rgb(243, 245, 247); display: inline !important;"><span style="font-family: calibri, arial, helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0);"> </span></span><a href="https://reviews.llvm.org/rG3a624c327adde62d075a4477e9bb9e6a2c186731" style="color: rgb(75, 77, 81); cursor: pointer; font-family: "Segoe UI", "Segoe UI Emoji", "Segoe UI Symbol", Lato, "Helvetica Neue", Helvetica, Arial, sans-serif; font-size: 13px; text-align: left; background-color: rgb(243, 245, 247);"><span style="font-family: calibri, arial, helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0);">rG3a624c327add</span></a><span style="font-family: calibri, arial, helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0); background-color: rgba(0, 0, 0, 0);">.</span></div>

<div id="appendonsend"></div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Florian Hahn <florian_hahn@apple.com><br>

<b>Sent:</b> 16 July 2020 13:27<br>

<b>To:</b> reviews+D83570+public+81ea3bf8c3fc81d9@reviews.llvm.org <reviews+D83570+public+81ea3bf8c3fc81d9@reviews.llvm.org><br>

<b>Cc:</b> Sjoerd Meijer <Sjoerd.Meijer@arm.com>; llvm-commits <llvm-commits@lists.llvm.org><br>

<b>Subject:</b> Re: [PATCH] D83570: [Matrix] Lowering pass should also run at O0</font>

<div> </div>

</div>

<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">

<div class="PlainText"><br>

<br>

> On Jul 16, 2020, at 12:18, Sjoerd Meijer via Phabricator <reviews@reviews.llvm.org> wrote:<br>

> <br>

> SjoerdMeijer added a comment.<br>

> <br>

> In D83570#2155389 <<a href="https://reviews.llvm.org/D83570#2155389">https://reviews.llvm.org/D83570#2155389</a>>, @fhahn wrote:<br>

> <br>

>> In D83570#2144734 <<a href="https://reviews.llvm.org/D83570#2144734">https://reviews.llvm.org/D83570#2144734</a>>, @SjoerdMeijer wrote:<br>

>> <br>

>>> Ah okay, cheers, missed that. Time to abandon this.<br>

>> <br>

>> <br>

>> I just committed a similar fix upstream, because there are some bots building the test-suite with -O0, to unblock them. The proper fix (run in lightweight lowering in the backend) will need a bit more time unfortunately.<br>

> <br>

> <br>

> Cheers.<br>

> And I will commit the test of this patch then as that still seems useful to me.<br>

> <br>

<br>

<br>

Sounds good to me, thanks!</div>

</span></font></div>

</body>

</html>