Re: git: 63ff982b17ee - main - simd.7: add scalar strrchr() for RISC-V to manpage

Hi Kib,

Am Sat, Nov 01, 2025 at 08:00:47PM +0200 schrieb Konstantin Belousov:
> On Sat, Nov 01, 2025 at 11:48:19AM +0100, Robert Clausecker wrote:
> > Hi kib,
> > 
> > Am Sat, Nov 01, 2025 at 05:39:49AM +0200 schrieb Konstantin Belousov:
> > > On Fri, Oct 31, 2025 at 12:48:28PM +0000, Robert Clausecker wrote:
> > > >  #include <machine/asm.h>
> > > >  
> > > > +        .weak   rindex
> > > > +        .set    rindex, strrchr
> > > > +
> > > Was this alias addition intended?
> > 
> > Yes, this is intentional.  See also lib/lib/{aarch64,amd64}/string/strrchr.S.
> 
> Ok, let me reformulate my question.
> Was the 'rindex' symbol exported from libc on riscv before your commit?
> 
> I remember that there was some effort to not provide 'obsoleted' exports
> from libc on the new arches like aarch64 and riscv.

As far as I know, yes.  lib/libc/strrchr.c provides the same alias.

Yours,
Robert Clausecker

-- 
()  ascii ribbon campaign - for an encoding-agnostic world
/\  - against html email  - against proprietary attachments