On Fri, 8 Apr 2022 14:42:34 GMT, Andrew Leonard <aleon...@openjdk.org> wrote:

>> Andrew Leonard has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Trigger checks
>
> I am not sure why without the explicit .file directive that the FILE symbol 
> in the ELF info contains an entry pointing to the .o object file, here's what 
> it was before:
> 
>  31712: 0000000000000000     0 FILE    LOCAL  DEFAULT  ABS 
> /home/andrew2/jdk/build/linux-aarch64-server-release/hotspot/variant-server/libjvm/objs/atomic_linux_aarch64.o
> 
> It's as if the lack of the .file has caused some tooling (linker?) to create 
> this entry using the .o file path.

@andrew-m-leonard 
> So I am thinking adding a .file directive to the .S file specifying the "full 
> source path" should do the trick thanks to -debug-prefix-map, I will do a 
> test..

But then you'll end up with that absolute path name embedded into `STT_FILE` in 
the object file. It's best to use the relative path name *and* employ path 
mapping.

@andrew-m-leonard 
> It's as if the lack of the .file has caused some tooling (linker?) to create 
> this entry using the .o file path.

I'm sure that's exactly what happens; the linker is the culprit.

-------------

PR: https://git.openjdk.java.net/jdk/pull/8124

Reply via email to