[Bug build/29454] New: dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Bug build/29454] New: dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

            Bug ID: 29454
           Summary: dl-cache.c:513: undefined reference to `strcpy' with
                    -Og or -O1 (doesn't happen with -O2)
           Product: glibc
           Version: 2.36
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: build
          Assignee: unassigned at sourceware dot org
          Reporter: Martin.Jansa at gmail dot com
                CC: carlos at redhat dot com
  Target Milestone: ---

When building glibc with -Og I've started to see "dl-cache.c:513: undefined
reference to `strcpy'" introduced in:

6b9006bfb0 x86: Move strcpy SSE2 implementation to multiarch/strcpy-sse2.S

x86_64-oe-linux-gcc (GCC) 12.1.0

x86_64-oe-linux-gcc  -m64 -march=core2 -mtune=core2 -msse3 -mfpmath=sse 
--sysroot=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/recipe-sysroot
-Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed
-fmacro-prefix-map=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0=/usr/src/debug/glibc/2.36-r0
                    
-fdebug-prefix-map=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0=/usr/src/debug/glibc/2.36-r0
                    
-fdebug-prefix-map=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/recipe-sysroot=
                    
-fdebug-prefix-map=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/recipe-sysroot-native=
 -Wl,-z,relro,-z,now -fuse-ld=bfd  -nostdlib -nostartfiles -r -o
/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/build-x86_64-oe-linux/elf/librtld.os
'-Wl,-('
/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/build-x86_64-oe-linux/elf/dl-allobjs.os
/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/build-x86_64-oe-linux/elf/rtld-libc.a
-lgcc '-Wl,-)' \
         
-Wl,-Map,/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/build-x86_64-oe-linux/elf/librtld.os.map
x86_64-oe-linux-gcc  -m64 -march=core2 -mtune=core2 -msse3 -mfpmath=sse 
--sysroot=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/recipe-sysroot
-Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed
-fmacro-prefix-map=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0=/usr/src/debug/glibc/2.36-r0
                    
-fdebug-prefix-map=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0=/usr/src/debug/glibc/2.36-r0
                    
-fdebug-prefix-map=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/recipe-sysroot=
                    
-fdebug-prefix-map=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/recipe-sysroot-native=
 -Wl,-z,relro,-z,now -fuse-ld=bfd  -nostdlib -nostartfiles -shared -o
/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/build-x86_64-oe-linux/elf/ld.so.new
             \
          -Wl,-z,relro -Wl,-z,defs -Wl,-z,now   \
          -Wl,-z,pack-relative-relocs \
         
/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/build-x86_64-oe-linux/elf/librtld.os
-Wl,--version-script=/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/build-x86_64-oe-linux/ld.map
             \
          -Wl,-soname=ld-linux-x86-64.so.2
/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/recipe-sysroot-native/usr/bin/x86_64-oe-linux/../../libexec/x86_64-oe-linux/gcc/x86_64-oe-linux/12.1.0/ld.bfd:
/OE/build/oe-core/tmp-glibc/work/core2-64-oe-linux/glibc/2.36-r0/build-x86_64-oe-linux/elf/librtld.os:
in function `_dl_load_cache_lookup':
/usr/src/debug/glibc/2.36-r0/git/elf/dl-cache.c:513: undefined reference to
`strcpy'
collect2: error: ld returned 1 exit status

There was another build issue when building with -Og or -O1 in bug 29249.

As work around I'm reverting these 3 commits (the middle one is just to avoid
conflicts with the top one, but 6b9006bfb0 is where the strcpy was first
reproducible:

04538ceb0d Revert "x86: Move strcpy SSE2 implementation to
multiarch/strcpy-sse2.S"
3e31c3f7c6 Revert "x86: Add support to build st{p|r}{n}{cpy|cat} with explicit
ISA level"
e5ec49260b Revert "Linux: Implement a useful version of _startup_fatal"

I don't understand this code at all, but from poking what changed with objdump
I've noticed that the symbol in rtld-strcpy-sse2.os is now uppercase:

objdump -x build-x86_64-oe-linux/string/rtld-strcpy-sse2.os | grep -i strcpy
build-x86_64-oe-linux/string/rtld-strcpy-sse2.os:     file format elf64-x86-64
build-x86_64-oe-linux/string/rtld-strcpy-sse2.os
0000000000000000 g     F .text  00000000000000ec STRCPY

while build from the commit just before 6b9006bfb0 shows:

objdump -x build-x86_64-oe-linux/string/rtld-strcpy-sse2.os | grep -i strcpy
build-x86_64-oe-linux/string/rtld-strcpy-sse2.os:     file format elf64-x86-64
build-x86_64-oe-linux/string/rtld-strcpy-sse2.os
0000000000000000 g     F .text  00000000000000ec strcpy

This might be also interesting, librtld.os.map diff shows only
elf/rtld-libc.a(rtld-stpcpy.os) now, while before it was showing
elf/rtld-libc.a(rtld-stpcpy-sse2.os) and elf/rtld-libc.a(rtld-strcpy-sse2.os)

--- build-x86_64-oe-linux.O2/elf/librtld.mk     2022-08-07 13:20:11.154223394
+0000
+++ build-x86_64-oe-linux.Og/elf/librtld.mk     2022-08-07 13:18:29.389217048
+0000
@@ -16,6 +16,7 @@
 rtld-string +=stpcpy.os
 rtld-string +=strchr.os
 rtld-string +=strcmp.os
+rtld-string +=strcpy.os
 rtld-string +=strcspn.os
 rtld-string +=strdup.os
 rtld-string +=strerrorname_np.os
@@ -112,6 +113,16 @@
 rtld-string +=strcmp.os
 rtld-string +=strcmp-sse4_2.os
 rtld-string +=strcmp.os
+rtld-string +=strcpy-avx2.os
+rtld-string +=strcpy.os
+rtld-string +=strcpy-avx2-rtm.os
+rtld-string +=strcpy.os
+rtld-string +=strcpy-evex.os
+rtld-string +=strcpy.os
+rtld-string +=strcpy-sse2.os
+rtld-string +=strcpy.os
+rtld-string +=strcpy-sse2-unaligned.os
+rtld-string +=strcpy.os
 rtld-string +=strcspn-sse4.os
 rtld-string +=strcspn.os
 rtld-string +=strlen-avx2.os

Replacing -Og with -O2 fixes the issue (but unlike bug 29249 "-O1
-fexpensive-optimizations" instead of "-Og" isn't enough here).

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

Martin Jansa <Martin.Jansa at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |goldstein.w.n at gmail dot com

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #1 from Martin Jansa <Martin.Jansa at gmail dot com> ---
diffoscope output when comparing elf/dl-cache.os built with -Og and O2.

|  /usr/src/debug/glibc/2.36-r0/git/elf/dl-cache.c:513
│ -     call   881 <_dl_load_cache_lookup+0xa1>
│ - R_X86_64_PLT32      memcpy-0x4
│ -     mov    %rax,%rdi
│ +     mov    %rbx,%rsi
│ +     mov    %r12,%rdi
│ +     call   7c9 <_dl_load_cache_lookup+0xaa>
│ + R_X86_64_PLT32      strcpy-0x4
│  /usr/src/debug/glibc/2.36-r0/git/elf/dl-cache.c:514
│ -     call   889 <_dl_load_cache_lookup+0xa9>
│ +     mov    %r12,%rdi
│ +     call   7d1 <_dl_load_cache_lookup+0xb2>
│   R_X86_64_PLT32      __strdup-0x4

And then it's also reproducible with dl-cache.os rebuilt using "-O2
-fno-builtin-strcpy"

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #2 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
Created attachment 14258
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14258&action=edit
Fix strcpy ifndef guard

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #3 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to Martin Jansa from comment #1)
> diffoscope output when comparing elf/dl-cache.os built with -Og and O2.
> 
> |  /usr/src/debug/glibc/2.36-r0/git/elf/dl-cache.c:513
> │ -     call   881 <_dl_load_cache_lookup+0xa1>
> │ - R_X86_64_PLT32      memcpy-0x4
> │ -     mov    %rax,%rdi
> │ +     mov    %rbx,%rsi
> │ +     mov    %r12,%rdi
> │ +     call   7c9 <_dl_load_cache_lookup+0xaa>
> │ + R_X86_64_PLT32      strcpy-0x4
> │  /usr/src/debug/glibc/2.36-r0/git/elf/dl-cache.c:514
> │ -     call   889 <_dl_load_cache_lookup+0xa9>
> │ +     mov    %r12,%rdi
> │ +     call   7d1 <_dl_load_cache_lookup+0xb2>
> │   R_X86_64_PLT32      __strdup-0x4
> 
> And then it's also reproducible with dl-cache.os rebuilt using "-O2
> -fno-builtin-strcpy"

Attached a patch can you test if that fixes it?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #4 from Martin Jansa <Martin.Jansa at gmail dot com> ---
Thanks for super fast reply, but the attached patch doesn't seem to fix it.

Also this "ifndef STPCPY" was introduced few commits later in:
49889fb256 x86: Add support to build st{p|r}{n}{cpy|cat} with explicit ISA
level

Cheers,

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #5 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to Martin Jansa from comment #4)
> Thanks for super fast reply, but the attached patch doesn't seem to fix it.
> 
> Also this "ifndef STPCPY" was introduced few commits later in:
> 49889fb256 x86: Add support to build st{p|r}{n}{cpy|cat} with explicit ISA
> level
> 
> Cheers,

What is the exact command you are using?
I tried:

```
rm -rf /home/noah/programs/opensource/glibc-dev/build; mkdir -p
/home/noah/programs/opensource/glibc-dev/build/glibc/; (cd
/home/noah/programs/opensource/glibc-dev/build/glibc/; unset LD_LIBRARY_PATH;
/home/noah/programs/opensource/glibc-dev/src/glibc/configure --prefix=/usr
CC="gcc -march=core2 -Og"; CXX="g++ -march=core2 -Og"; make -j3 --silent;);
```

And wasn't able to reproduce this.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #6 from Martin Jansa <Martin.Jansa at gmail dot com> ---
Created attachment 14259
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14259&action=edit
config.status

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #7 from Martin Jansa <Martin.Jansa at gmail dot com> ---
I'm using OpenEmbedded cross-compilation, whole log is here:
http://errors.yoctoproject.org/Errors/Details/663898/

The flags are used from config.status (attached) generated in do_configure,
then it runs 'make PARALLELMFLAGS="-j 70 -l 140" SHELL=/bin/bash KSHELL=/bin/sh
"$@"' in the build-x86_64-oe-linux created by do_configure.

I'll try to reproduce it in native build on gentoo (which will be closer to
what you're using).

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #8 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to Martin Jansa from comment #6)
> Created attachment 14259 [details]
> config.status

Thanks.

H.J do you know why rtld-strcpy-sse2 is being built at all?

I was under the impression it would only be built with an rtld-... hook.
strcpy-sse2.S should not be built for rtld/non-multiarch directly, it should go
through
sysdeps/x86_64/strcpy.S

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #9 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to Martin Jansa from comment #7)
> I'm using OpenEmbedded cross-compilation, whole log is here:
> http://errors.yoctoproject.org/Errors/Details/663898/
> 
> The flags are used from config.status (attached) generated in do_configure,
> then it runs 'make PARALLELMFLAGS="-j 70 -l 140" SHELL=/bin/bash
> KSHELL=/bin/sh "$@"' in the build-x86_64-oe-linux created by do_configure.
> 
> I'll try to reproduce it in native build on gentoo (which will be closer to
> what you're using).

Thanks.

I see whats causing this in the code but am not sure why it's occurring. H.J
may have an idea.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

Noah Goldstein <goldstein.w.n at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hjl.tools at gmail dot com

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #10 from Martin Jansa <Martin.Jansa at gmail dot com> ---
I was able to reproduce it in native build after exporting CFLAGS="-Og"

my build.sh attached.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #11 from Martin Jansa <Martin.Jansa at gmail dot com> ---
Created attachment 14260
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14260&action=edit
build.sh

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

Martin Jansa <Martin.Jansa at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #14260|application/x-shellscript   |text/plain
          mime type|                            |

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

Martin Jansa <Martin.Jansa at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #14259|application/x-shellscript   |text/plain
          mime type|                            |

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #12 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to Martin Jansa from comment #11)
> Created attachment 14260 [details]
> build.sh

Thanks was able to reproduce with that.

Attached is another patch that I think fixes it. Now fail at 
```
/usr/bin/ld:
/home/noah/programs/opensource/glibc-dev/build/glibc/libc.a(libc-tls.o): in
function `__libc_setup_tls':
libc-tls.c:(.text+0x2f2): undefined reference to `_startup_fatal_not_constant'
collect2: error: ld returned 1 exit status
```

which I think is a different issue.

H.J I will post this patch (and the other one for the ifdef tomorrow morning).
Do you know why this was necessary for the non-optimized build?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #13 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
Created attachment 14261
  --> https://sourceware.org/bugzilla/attachment.cgi?id=14261&action=edit
RTLD strcpy hook

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #14 from Martin Jansa <Martin.Jansa at gmail dot com> ---
> Attached is another patch that I think fixes it.

Yes, it fixes it for me as well in native as well as OpenEmbedded build, thanks
a lot for quick update!

> undefined reference to `_startup_fatal_not_constant'
> which I think is a different issue.

Yes, I'm seeing this one as well, that's bug 29249

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #15 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to Martin Jansa from comment #14)
> > Attached is another patch that I think fixes it.
> 
> Yes, it fixes it for me as well in native as well as OpenEmbedded build,
> thanks a lot for quick update!

Great. Will keep this open until changes reach master / are potentially
backported.

Thanks for the report.
> 
> > undefined reference to `_startup_fatal_not_constant'
> > which I think is a different issue.
> 
> Yes, I'm seeing this one as well, that's bug 29249

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[hjl.tools at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #16 from H.J. Lu <hjl.tools at gmail dot com> ---
(In reply to Noah Goldstein from comment #12)

> which I think is a different issue.
> 
> H.J I will post this patch (and the other one for the ifdef tomorrow
> morning). Do you know why this was necessary for the non-optimized build?

Some parts of glibc must be compiled with optimization.  If there is foo.[cS]
under sysdeps/x86_64/multiarch, the one in sysdeps/x86_64 will be ignored.
Does GCC inline strcpy with -O2 in dl-cache.c?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #17 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to H.J. Lu from comment #16)
> (In reply to Noah Goldstein from comment #12)
> 
> > which I think is a different issue.
> > 
> > H.J I will post this patch (and the other one for the ifdef tomorrow
> > morning). Do you know why this was necessary for the non-optimized build?
> 
> Some parts of glibc must be compiled with optimization.  If there is foo.[cS]
> under sysdeps/x86_64/multiarch, the one in sysdeps/x86_64 will be ignored.
> Does GCC inline strcpy with -O2 in dl-cache.c?

Yeah the usage is:

```
  char *temp;
  temp = alloca (strlen (best) + 1);
  strcpy (temp, best);
  return __strdup (temp);
```

Which gets converted to `memcpy` (because of already computed strlen) in
strlen_pass::handle_builtin_strcpy.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #18 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
Patch posted. Ended up just replacing the strcpy call with memcpy directly so
we don't need to include strcpy in the rtld build.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[yann at droneaud dot fr]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

Yann Droneaud <yann at droneaud dot fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |yann at droneaud dot fr

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[sam at gentoo dot org]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

Sam James <sam at gentoo dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sam at gentoo dot org

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #19 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to Noah Goldstein from comment #18)
> Patch posted. Ended up just replacing the strcpy call with memcpy directly
> so we don't need to include strcpy in the rtld build.

Pushed patch:
https://sourceware.org/git/?p=glibc.git;a=commit;h=483cfe1a6a33d6335b1901581b41040d2d412511

H.J should we backport this?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[hjl.tools at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #20 from H.J. Lu <hjl.tools at gmail dot com> ---
(In reply to Noah Goldstein from comment #19)
> (In reply to Noah Goldstein from comment #18)
> > Patch posted. Ended up just replacing the strcpy call with memcpy directly
> > so we don't need to include strcpy in the rtld build.
> 
> Pushed patch:
> https://sourceware.org/git/?p=glibc.git;a=commit;
> h=483cfe1a6a33d6335b1901581b41040d2d412511
> 
> H.J should we backport this?

It doesn't look like a regression.  Please ask it on the mailing list.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[Martin.Jansa at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #21 from Martin Jansa <Martin.Jansa at gmail dot com> ---
Please backport it, this part was building with -Og in 2.35 and with 2.36 it
doesn't, so I would stay it's an regression, right?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug build/29454] dl-cache.c:513: undefined reference to `strcpy' with -Og or -O1 (doesn't happen with -O2)

[goldstein.w.n at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=29454

--- Comment #22 from Noah Goldstein <goldstein.w.n at gmail dot com> ---
(In reply to Martin Jansa from comment #21)
> Please backport it, this part was building with -Og in 2.35 and with 2.36 it
> doesn't, so I would stay it's an regression, right?

backported:
https://sourceware.org/git/?p=glibc.git;a=commit;h=302bc33bc53c787da6e74162a7092e9c0fb964a8

-- 
You are receiving this mail because:
You are on the CC list for the bug.