[llvm] [Instcombine] Fix crash in foldMinimumMaximumSharedOp. (PR #173705)

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 26 23:08:18 PST 2025


================
@@ -0,0 +1,16 @@
+; RUN: opt -passes=instcombine -S %s -o - | FileCheck %s
----------------
jlebar wrote:

Are you sure that's a good idea?  That would change this reproducer into a "change-detector test".  Change-detector tests are fragile by construction.  And when they do fail, it's not clear whether the new IR is "acceptable" / "correct".

It's been my practice to write tests that check for "correctness" or "no regression" and no more.  In this case "no regression" would mean "LLVM doesn't crash".

WDYT?

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


More information about the llvm-commits mailing list