[all-commits] [llvm/llvm-project] 112e3d: Add a pre-commit test for D124426

Juneyoung Lee via All-commits all-commits at lists.llvm.org
Sat Apr 30 04:51:55 PDT 2022


  Branch: refs/heads/maiin
  Home:   https://github.com/llvm/llvm-project
  Commit: 112e3d86870d214bfa27ce18bfa99eb8197e2bc7
      https://github.com/llvm/llvm-project/commit/112e3d86870d214bfa27ce18bfa99eb8197e2bc7
  Author: Juneyoung Lee <aqjune at gmail.com>
  Date:   2022-04-30 (Sat, 30 Apr 2022)

  Changed paths:
    A llvm/test/Transforms/InstCombine/select-cmp.ll

  Log Message:
  -----------
  Add a pre-commit test for D124426


  Commit: 40a2e35599b5e5fa0dd93ac4edf4bf02ee636f6e
      https://github.com/llvm/llvm-project/commit/40a2e35599b5e5fa0dd93ac4edf4bf02ee636f6e
  Author: Juneyoung Lee <aqjune at gmail.com>
  Date:   2022-04-30 (Sat, 30 Apr 2022)

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

  Log Message:
  -----------
  [InstCombine] Remove the undef-related workaround code in visitSelectInst

This patch removes an old hack in visitSelectInst that was written to avoid miscompilation bugs in loop unswitch.
(Added via https://reviews.llvm.org/D35811)

The legacy loop unswitch pass will be removed after D124376, and the new simple loop unswitch pass correctly uses freeze to avoid introducing UB after D124252.

Since the hack is not necessary anymore, this patch removes it.

Reviewed By: nikic

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


Compare: https://github.com/llvm/llvm-project/compare/112e3d86870d%5E...40a2e35599b5


More information about the All-commits mailing list