mk: avoid combining -r and -export-dynamic linker options
authorRafael Ávila de Espíndola <espindola@scylladb.com>
Thu, 12 Dec 2019 17:53:12 +0000 (09:53 -0800)
committerThomas Monjalon <thomas@monjalon.net>
Sun, 19 Jan 2020 23:00:28 +0000 (00:00 +0100)
commit5a352a3a98e3297555cb08d2db6e45a93ce7e058
treea203af6275ac468a74c1af469fdc144a201b13fe
parent9c5d218462ef88483ea023fcd38a3221f7f0b4bf
mk: avoid combining -r and -export-dynamic linker options

Running ld with -r switches the linker to a very special mode where
some other linker options don't make sense.

In particular, -export-dynamic normally requires that all global
symbols be included in the dynamic symbol table, but a .o file doesn't
even have a dynamic symbol table.

When given both options it looks like the gnu linker just ignores
-export-dynamic.

Unfortunately some versions of lld (https://lld.llvm.org/) have a bug
that causes it to try to create a dynamic symbol table in the output
.o file and ends up corrupting it
(https://bugs.llvm.org/show_bug.cgi?id=43552). Current (git) version
of lld now issues an error.

This patch filters out -export-dynamic from $(LDFLAGS) when using
-r. With this patch I can build dpdk with lld.

Fixes: 3d781ca32874 ("mk: do post processing on objects that register a driver")
Cc: stable@dpdk.org
Signed-off-by: Rafael Ávila de Espíndola <espindola@scylladb.com>
mk/internal/rte.compile-pre.mk