[clang] [llvm] [Arm] Regenerate tests (NFC) (PR #121801)

David Green via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 6 09:17:21 PST 2025


davemgreen wrote:

> This patch adds instcombine to some tests that were passing

This patch sounds good, but the intent of the clang tests is that they should not be dependant on the mid-end optimizations in llvm. At least for fast-changing parts like instcombine, where you don't want to have to update a thousand clang tests every time instcombine learns a new trick. The clang tests should ideally be testing the clang output.

Can the instcombines be replaced with something simpler like dce or maybe instsimplify? It might be OK with just mem2reg.

https://github.com/llvm/llvm-project/pull/121801


More information about the llvm-commits mailing list