695719 - Pixman ARM assembly symbols tweaks

Status: RESOLVED WONTFIX

(Core :: Graphics, defect)

Description

[Mike Hommey [:glandium]]

The pixman ARM assembly symbols all end up in the .text section even with -ffunction-sections, and have a size of 0, which makes them invisible to e.g. valgrind.

Comment 1

[Mike Hommey [:glandium]]

Attachment: Pixman ARM assembly symbols tweaks

It would be better if the .section tweaks depended on -ffunction-sections being there, but there's no way to know :-/

I'll need to check if that has some unwanted side effects when not building with -ffunction-sections.

Comment 2

[Siarhei Siamashka]

Is using -ffunction-sections option and putting each function into its own section really necessary? Based on my quick test (admittedly done in linux, not android), looks like just the part of your patch adding .size directives is enough to solve valgrind compatibility problems.

Could you please also send your patch to the pixman mailing list or alternatively file a bug with the patch attached at https://bugs.freedesktop.org/ ?

PS. to play nicer with various debugging tools, also adding unwind tables generation directives to the assembly code might be useful sooner or later.

Comment 3

[Mike Hommey [:glandium]]

(In reply to Siarhei Siamashka from comment #2)
> Is using -ffunction-sections option and putting each function into its own
> section really necessary?

It's useful for function/section reordering.

> Could you please also send your patch to the pixman mailing list or
> alternatively file a bug with the patch attached at
> https://bugs.freedesktop.org/ ?

I will, when i figure whether the function section blows things up without -ffunction-sections