[all-commits] [llvm/llvm-project] db40f9: [InstCombine] add test for logical-ands to select; ...

Sanjay Patel via All-commits all-commits at lists.llvm.org
Sat Oct 22 07:44:35 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: db40f9b77417aa0d5be2200a4972e5f696ab7c34
      https://github.com/llvm/llvm-project/commit/db40f9b77417aa0d5be2200a4972e5f696ab7c34
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2022-10-22 (Sat, 22 Oct 2022)

  Changed paths:
    M llvm/test/Transforms/InstCombine/select-safe-transforms.ll

  Log Message:
  -----------
  [InstCombine] add test for logical-ands to select; NFC


  Commit: 8628e6df70008e3eba86aee2a128b2e9b1f7cc26
      https://github.com/llvm/llvm-project/commit/8628e6df70008e3eba86aee2a128b2e9b1f7cc26
  Author: Sanjay Patel <spatel at rotateright.com>
  Date:   2022-10-22 (Sat, 22 Oct 2022)

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

  Log Message:
  -----------
  [InstCombine] use freeze to enable poison-safe logic->select fold

Without a freeze, this transform can leak poison to the output:
https://alive2.llvm.org/ce/z/GJuF9i

This makes the transform as uniform as possible, and it can help
reduce patterns like issue #58313 (although that particular
example probably still needs another transform).

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


Compare: https://github.com/llvm/llvm-project/compare/0cf6515e27d4...8628e6df7000


More information about the All-commits mailing list