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

Robin Lindner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 14 01:05:53 PDT 2022


deeprobin added a comment.

In D6407#3377887 <https://reviews.llvm.org/D6407#3377887>, @RKSimon wrote:

> Sorry for not replying (I tend to assume any comments on ancient phabs are spambots).
>
> Developers are human too - they might not intentionally do many things, but they still crop up!

All is well :D

> IIRC (8 years...........) as well as PR15782, this particular case turned up in some ABGR -> RGBA pixel code swizzling. I can imagine it potentially happening in network 'ntohl' style ops as well.

Thank you very much. I think that helps me already.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D6407



More information about the llvm-commits mailing list