[all-commits] [llvm/llvm-project] af4367: [InstCombine] Skip foldFBinOpOfIntCastsFromSign fo...

[Nikita Popov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: af4367a6db633f979a3f8c565335d279a9c1d976
      https://github.com/llvm/llvm-project/commit/af4367a6db633f979a3f8c565335d279a9c1d976
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2025-10-13 (Mon, 13 Oct 2025)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstructionCombining.cpp
    M llvm/test/Transforms/InstCombine/add-sitofp.ll
    M llvm/test/Transforms/InstCombine/binop-itofp.ll

  Log Message:
  -----------
  [InstCombine] Skip foldFBinOpOfIntCastsFromSign for vector ops (#162804)

Converting a vector float op into a vector int op may be non-profitable,
especially for targets where the float op for a given type is legal, but
the integer op is not.

We could of course also try to address this via a reverse transform in
the backend, but I don't think it's worth the bother, given that vectors
were never the intended use case for this transform in the first place.

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



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications