[clang] [lld] [llvm] [RISCV] Make M imply Zmmul (PR #95070)

2024-06-21 Thread Philip Reames via cfe-commits
preames wrote: Given the concern about breaking configurations w/no-integrated-as and older binutils, can someone summarize here which versions of binutils are known to work/not work after this change? This will likely become the key search result for such breakage, and having it well

[clang] [lld] [llvm] [RISCV] Make M imply Zmmul (PR #95070)

2024-06-20 Thread Jianjian Guan via cfe-commits
https://github.com/jacquesguan closed https://github.com/llvm/llvm-project/pull/95070 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [lld] [llvm] [RISCV] Make M imply Zmmul (PR #95070)

2024-06-20 Thread Craig Topper via cfe-commits
https://github.com/topperc approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/95070 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [lld] [llvm] [RISCV] Make M imply Zmmul (PR #95070)

2024-06-11 Thread Jianjian Guan via cfe-commits
jacquesguan wrote: > We intentionally didn't do this because it makes LLVM not work with binutils > that doesn't know about Zmmul. Maybe enough time has passed that it is no > longer an issue? Since Zmmul was ratified at june 2022, I think it's OK to update it now.

[clang] [lld] [llvm] [RISCV] Make M imply Zmmul (PR #95070)

2024-06-11 Thread Jianjian Guan via cfe-commits
https://github.com/jacquesguan updated https://github.com/llvm/llvm-project/pull/95070 >From dfbbcec9140f625309f78ca98c45b26b89fb71b1 Mon Sep 17 00:00:00 2001 From: Jianjian GUAN Date: Thu, 6 Jun 2024 14:21:12 +0800 Subject: [PATCH] [RISCV] Make M imply Zmmul According to the spec, M implies