[all-commits] [llvm/llvm-project] 5643be: [DAG] Enable foldSelectOfBinops on select(setcc(), ...

Simon Pilgrim via All-commits all-commits at lists.llvm.org
Sun Jul 18 10:39:29 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5643be96bc1fd6e8b00aa25b0bb274511d6b1100
      https://github.com/llvm/llvm-project/commit/5643be96bc1fd6e8b00aa25b0bb274511d6b1100
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-07-18 (Sun, 18 Jul 2021)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    M llvm/test/CodeGen/AMDGPU/idiv-licm.ll
    M llvm/test/CodeGen/X86/combine-shl.ll

  Log Message:
  -----------
  [DAG] Enable foldSelectOfBinops on select(setcc(),binop(),binop()) calls


  Commit: fd7a54c709083f79d8d1062156b9e386a506b189
      https://github.com/llvm/llvm-project/commit/fd7a54c709083f79d8d1062156b9e386a506b189
  Author: Simon Pilgrim <llvm-dev at redking.me.uk>
  Date:   2021-07-18 (Sun, 18 Jul 2021)

  Changed paths:
    M llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
    M llvm/test/CodeGen/X86/combine-shl.ll

  Log Message:
  -----------
  [DAG] DAGCombiner::foldSelectOfBinops - propagate the common flags to the merged binop

As discussed on D106058 - we were failing to keep the common flags. This matches the behaviour in InstCombinerImpl::foldSelectOpOp.


Compare: https://github.com/llvm/llvm-project/compare/954711ed8f0d...fd7a54c70908


More information about the All-commits mailing list