On 6/13/24 13:02, Jeff Law wrote:


On 6/12/24 5:20 PM, Patrick O'Neill wrote:
Binutils 2.42 and before don't support Zaamo/Zalrsc. Add a configure
check to prevent emitting Zaamo/Zalrsc in the arch string when the
assember does not support it.

gcc/ChangeLog:

    * common/config/riscv/riscv-common.cc
      (riscv_subset_list::to_string): Skip zaamo/zalrsc when not
      supported by the assembler.
    * config.in: Regenerate.
    * configure: Regenerate.
    * configure.ac: Add zaamo/zalrsc assmeber check.
OK.

It looks like you've got some unexpected diff fragmets in configure -- all the LARGE_OFF_T stuff.  They look OK to me, but something like that is usually a sign of different autoconf versions.   I wouldn't lose any sleep if you left them as-is or removed those hunks before committing.

jeff

Removed the hunks and committed.
Sent the committed version to the list for the archiver.

I'll rebase the promotion RFC [1] on top and resolve the warning that Andreas Schwab noticed.

Patrick

[1]: https://patchwork.sourceware.org/project/gcc/patch/20240613233059.1451117-1-patr...@rivosinc.com/

Reply via email to