On 11/17/20 12:57 AM, Richard Biener via Gcc-patches wrote:
> On Tue, Nov 17, 2020 at 3:19 AM Eugene Rozenfeld
> <eugene.rozenf...@microsoft.com> wrote:
>> Thank you for the review Richard!
>>
>> I re-worked the patch based on your suggestions (attached).
>> I made the change to reuse the first bit_xor in both patterns and I added :s
>> to the last xor in the first pattern.
>> For the second pattern I didn't add :s because I think the simplification is
>> beneficial even if the second or third bit_xor has more than one use since
>> we are simplifying them to just a single operand (@2). If that is incorrect,
>> please explain why.
> Ah, true, that's correct.
>
> The patch is OK.
I've installed this on the trunk.
Eugene, if you're going to contribute regularly you should probably go
ahead and get commit privs so that you can commit ACK's patches
yourself. There should be a link to a form from this page:
https://gcc.gnu.org/gitwrite.html
Jeff
