[all-commits] [llvm/llvm-project] 8378f1: [InstCombine] Remove adjustMinMax() fold (PR62088)

Nikita Popov via All-commits all-commits at lists.llvm.org
Tue May 30 07:06:56 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8378f1f4cdc8922e4f0409cabff25e0fef517bfa
      https://github.com/llvm/llvm-project/commit/8378f1f4cdc8922e4f0409cabff25e0fef517bfa
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2023-05-30 (Tue, 30 May 2023)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineSelect.cpp
    M llvm/test/Transforms/InstCombine/select.ll

  Log Message:
  -----------
  [InstCombine] Remove adjustMinMax() fold (PR62088)

This fold is buggy if the constant adjustment overflows.
Additionally, since we now canonicalize to min/max intrinsics,
the constants picked here don't actually matter, as long as SPF
still recognizes the pattern.

Fixes https://github.com/llvm/llvm-project/issues/62088.




More information about the All-commits mailing list