[PATCH] D6407: [InstCombine] Minor optimization for bswap with binary ops

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D6407#3377769 <https://reviews.llvm.org/D6407#3377769>, @deeprobin wrote:

> @xbolva00 @RKSimon
>
>> 8 year old patch…
>
> Yes I would be interested in the real-world case behind this.
>
>>>> No developer will …
>>
>> Very strong words :D
>
> The question is whether this is a niche optimization or whether there is some common code pattern that justifies that this is not a niche optimization.

Perhaps it would be more productive to approach this from another angle - what prompted you to bring this up?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D6407/new/

https://reviews.llvm.org/D6407