[PATCH] D62257: [DAGCombiner][X86][AArch64] (x - C) + y -> (x + y) - C fold

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 22 08:25:53 PDT 2019


lebedev.ri added a comment.

In D62257#1512176 <https://reviews.llvm.org/D62257#1512176>, @RKSimon wrote:

> Its rather annoying that the DAGCombiner::visitSUB limits the (sub x, c) -> (add x, -c) fold to non-vectors


I suppose i could take a look at that, but i'd like to cram out the remaining patches for `sink-addsub-of-const.ll`.
And the follow-ups - preservation of `neg`, creation of `neg` (no new ISD opcode), thus fixing PR41952 <https://bugs.llvm.org/show_bug.cgi?id=41952>.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62257





More information about the llvm-commits mailing list