<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body>
<p style="font-family:Arial;font-size:10pt;color:#0000FF;margin:5pt;" align="Left">
[AMD Official Use Only]<br>
</p>
<br>
<div>
<div dir="auto" style="direction: ltr; margin: 0; padding: 0; font-family: sans-serif; font-size: 11pt; color: black; ">
That's too late now, but the second transformation was only correct with no undef inputs. I don't believe that has really affected anyone practically though.<span id="ms-outlook-android-cursor"></span></div>
<div><br>
</div>
<div id="ms-outlook-mobile-signature">
<div dir="auto">Stas</div>
</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> Thursday, November 25, 2021 2:44:43 AM<br>
<b>To:</b> Mekhanoshin, Stanislav <Stanislav.Mekhanoshin@amd.com>; Stanislav Mekhanoshin <llvmlistbot@llvm.org><br>
<b>Cc:</b> llvm-commits@lists.llvm.org <llvm-commits@lists.llvm.org><br>
<b>Subject:</b> Re: [llvm] 9300b13 - Revert "[InstCombine] (~(a | b) & c) | ~(c | (a ^ b)) -> ~((a | b) & (c | (b ^ a)))"</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">[CAUTION: External Email]<br>
<br>
Hi,<br>
<br>
> On Nov 24, 2021, at 19:14, Stanislav Mekhanoshin via llvm-commits <llvm-commits@lists.llvm.org> wrote:<br>
><br>
><br>
> Author: Stanislav Mekhanoshin<br>
> Date: 2021-11-24T11:14:52-08:00<br>
> New Revision: 9300b133c80dfc1a15135f9fb9c5afaf8a6805f8<br>
><br>
> URL: <a href="https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fllvm%2Fllvm-project%2Fcommit%2F9300b133c80dfc1a15135f9fb9c5afaf8a6805f8&data=04%7C01%7CStanislav.Mekhanoshin%40amd.com%7C8dbf308f7c41432b276e08d9b0009912%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637734338899105891%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=4YjURD%2B195eCOiOZB6QjhmRLCHU2yMJxFmWHeH1jIak%3D&reserved=0">
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fllvm%2Fllvm-project%2Fcommit%2F9300b133c80dfc1a15135f9fb9c5afaf8a6805f8&data=04%7C01%7CStanislav.Mekhanoshin%40amd.com%7C8dbf308f7c41432b276e08d9b0009912%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637734338899105891%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=4YjURD%2B195eCOiOZB6QjhmRLCHU2yMJxFmWHeH1jIak%3D&reserved=0</a><br>
> DIFF: <a href="https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fllvm%2Fllvm-project%2Fcommit%2F9300b133c80dfc1a15135f9fb9c5afaf8a6805f8.diff&data=04%7C01%7CStanislav.Mekhanoshin%40amd.com%7C8dbf308f7c41432b276e08d9b0009912%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637734338899105891%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=ZV%2FlAqavxahKeHKZ7w4YBYtHjU0L5RBbEwb8BE6qqVk%3D&reserved=0">
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fllvm%2Fllvm-project%2Fcommit%2F9300b133c80dfc1a15135f9fb9c5afaf8a6805f8.diff&data=04%7C01%7CStanislav.Mekhanoshin%40amd.com%7C8dbf308f7c41432b276e08d9b0009912%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637734338899105891%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=ZV%2FlAqavxahKeHKZ7w4YBYtHjU0L5RBbEwb8BE6qqVk%3D&reserved=0</a><br>
><br>
> LOG: Revert "[InstCombine] (~(a | b) & c) | ~(c | (a ^ b)) -> ~((a | b) & (c | (b ^ a)))"<br>
><br>
> This reverts commit c407769f5e6c81d56de0b251aed5750a16a7651c.<br>
<br>
<br>
It would be great if you could include a reason for the revert in the commit message, so it is a bit easier to see what issue the revert addresses for people following along.<br>
<br>
Cheers,<br>
Florian<br>
</div>
</span></font></div>
</div>
</body>
</html>