[all-commits] [llvm/llvm-project] d8e0e1: [InstCombine] Simplify addends reordering logic

Daniil Kovalev via All-commits all-commits at lists.llvm.org
Tue Jan 18 05:01:40 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d8e0e125a2ff03335838039f5d33ecd61dfec654
      https://github.com/llvm/llvm-project/commit/d8e0e125a2ff03335838039f5d33ecd61dfec654
  Author: Daniil Kovalev <daniil at kovalev.website>
  Date:   2022-01-18 (Tue, 18 Jan 2022)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineAddSub.cpp
    M llvm/test/Transforms/Reassociate/fast-basictest.ll

  Log Message:
  -----------
  [InstCombine] Simplify addends reordering logic

Previously some constants were not pushed to the top of the resulting
expression tree as intended by the algorithm. We can remove the logic
from simplifyFAdd and rely on SimplifyAssociativeOrCommutative to do
that.

Differential Revision: https://reviews.llvm.org/D117302




More information about the All-commits mailing list