[PATCH] D86798: [EarlyCSE] fold commutable intrinsics

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 28 12:57:15 PDT 2020


nikic added a comment.

In D86798#2245116 <https://reviews.llvm.org/D86798#2245116>, @spatel wrote:

> Or I can just leave the 'fix' intrinsics off of the list here for now. Do you have a preference?

Yeah, I think it would be best to drop the fixpoint intrinsics for now, as they aren't commutative in the usual sense and we won't be able to exploit that in the current implementation anyway.


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

https://reviews.llvm.org/D86798



More information about the llvm-commits mailing list