[all-commits] [llvm/llvm-project] 59003d: [InstCombine] Add tests for expanding `foldSelectV...

goldsteinn via All-commits all-commits at lists.llvm.org
Thu Jun 6 18:03:26 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 59003d714a9447bf03e0ce25ad4492290e80b110
      https://github.com/llvm/llvm-project/commit/59003d714a9447bf03e0ce25ad4492290e80b110
  Author: Noah Goldstein <goldstein.w.n at gmail.com>
  Date:   2024-06-06 (Thu, 06 Jun 2024)

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

  Log Message:
  -----------
  [InstCombine] Add tests for expanding `foldSelectValueEquivalence`; NFC


  Commit: 7e7c29ba087e38056b91f1d783db0883dcc33ef7
      https://github.com/llvm/llvm-project/commit/7e7c29ba087e38056b91f1d783db0883dcc33ef7
  Author: Noah Goldstein <goldstein.w.n at gmail.com>
  Date:   2024-06-06 (Thu, 06 Jun 2024)

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

  Log Message:
  -----------
  [InstCombine] Improve coverage of `foldSelectValueEquivalence` for constants

We don't need the `noundef` check if the new simplification is a
constant.

This cleans up regressions from folding multiuse:
    `(icmp eq/ne (sub/xor x, y), 0)` -> `(icmp eq/ne x, y)`.

Closes #88298


Compare: https://github.com/llvm/llvm-project/compare/f42025c2178b...7e7c29ba087e

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


More information about the All-commits mailing list