[PATCH] D48529: [DAGCombiner][X86][PowerPC][ARM][AMDGPU][Sparc] Replace (sub C, X) with (xor X, C) if there is no chance of a borrow during the subtraction.
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 25 08:35:28 PDT 2018
spatel added reviewers: nemanjai, kbarton, sfertile, syzaara.
spatel added a comment.
Looks like a generally good fold to me - prefer bitwise logic because that's the cheapest ALU op possible.
Adding some PowerPC reviewers to see if they're ok with the regression or if that must be fixed first.
Repository:
rL LLVM
https://reviews.llvm.org/D48529
More information about the llvm-commits
mailing list