[clang] [llvm] InstCombine: Order shufflevector operands by complexity (PR #113212)

Eli Friedman via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 21 13:31:14 PDT 2024


efriedma-quic wrote:

Please push the revised clang tests as a separate patch.

This idea here makes sense, I guess, but I'm a little worried about unexpected side-effects... not sure how reliably the backend handles commuted masks, and I don't think we have any testing infrastructure that would catch that sort of issue.

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


More information about the llvm-commits mailing list