This is an archive of the discontinued LLVM Phabricator instance.

[mips] Removed the SHF_ALLOC flag from the .pdr section.
ClosedPublic

Authored by s.egerton on Feb 12 2016, 6:31 AM.

Details

Summary

This section is used for debug information and has no need to be
in memory at runtime. With this patch, LLVM now emits the same flags as
the GNU assembler. This patch also fixes an error when compiling
the Linux kernel, The error is that there are relocations within the
.pdr section in a VDSO.

Diff Detail

Event Timeline

s.egerton updated this revision to Diff 47798.Feb 12 2016, 6:31 AM
s.egerton retitled this revision from to [mips] Made the .pdr section not allocatable..
s.egerton updated this object.
s.egerton added reviewers: dsanders, vkalintiris.
s.egerton added a subscriber: llvm-commits.
dsanders accepted this revision.Feb 12 2016, 7:07 AM
dsanders edited edge metadata.

LGTM with a couple tweaks to the commit message.

[mips] Made the .pdr section not allocatable.
This section is used for debug information and has no need to be
allocatable. This also causes errors when compiling the Linux kernel.

We should probably mention SHF_ALLOC on the first subject line.

Could you explain what kind of errors it fixes? It should be enough to mention eliminating relocations in a VDSO and that we now emit the same flags as GAS.

This revision is now accepted and ready to land.Feb 12 2016, 7:07 AM
s.egerton retitled this revision from [mips] Made the .pdr section not allocatable. to [mips] Removed the SHF_ALLOC flag from the .pdr section..Feb 12 2016, 7:25 AM
s.egerton updated this object.
s.egerton edited edge metadata.
s.egerton closed this revision.Feb 15 2016, 1:38 AM