[all-commits] [llvm/llvm-project] 3236d9: [InstCombine] Add test for `(fmul (sitfp x), 0)`; NFC

[goldsteinn via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3236d974992580d8a346b22720d9d290fbd5439f
      https://github.com/llvm/llvm-project/commit/3236d974992580d8a346b22720d9d290fbd5439f
  Author: Noah Goldstein <goldstein.w.n at gmail.com>
  Date:   2024-03-14 (Thu, 14 Mar 2024)

  Changed paths:
    M llvm/test/Transforms/InstCombine/binop-itofp.ll

  Log Message:
  -----------
  [InstCombine] Add test for `(fmul (sitfp x), 0)`; NFC


  Commit: 70d0ebb279e42902f06ab621777935b09e82cc49
      https://github.com/llvm/llvm-project/commit/70d0ebb279e42902f06ab621777935b09e82cc49
  Author: Noah Goldstein <goldstein.w.n at gmail.com>
  Date:   2024-03-14 (Thu, 14 Mar 2024)

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

  Log Message:
  -----------
  [InstCombine] Fix behavior for `(fmul (sitfp x), 0)`

Bug was introduced in #82555

We where missing check that the constant was non-zero for signed + mul
transform.

Closes #85298


Compare: https://github.com/llvm/llvm-project/compare/1510473908e0...70d0ebb279e4

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