[clang] [llvm] [BPF] Fix linking issues in static map initializers (PR #91310)
via llvm-commits
llvm-commits at lists.llvm.org
Tue May 14 13:18:53 PDT 2024
================
@@ -1950,8 +1950,22 @@ ConstantLValueEmitter::tryEmitBase(const APValue::LValueBase &base) {
if (D->hasAttr<WeakRefAttr>())
return CGM.GetWeakRefReference(D).getPointer();
- if (auto FD = dyn_cast<FunctionDecl>(D))
- return CGM.GetAddrOfFunction(FD);
+ if (auto FD = dyn_cast<FunctionDecl>(D)) {
+ auto *C = CGM.GetAddrOfFunction(FD);
+
+ // we don't normally emit debug info for extern fns referenced via
+ // variable initialisers; BPF needs it since it generates BTF from
+ // debug info and bpftool demands BTF for every symbol linked
+ if (CGM.getTarget().getTriple().isBPF() && FD->getStorageClass() == SC_Extern) {
----------------
yonghong-song wrote:
@mejedi @efriedma-quic Yes, we only want referenced functions/variables. The support for external variable has been implemented here https://github.com/llvm/llvm-project/commit/d77ae1552fc21a9f3877f3ed7e13d631f517c825. Thanks!
https://github.com/llvm/llvm-project/pull/91310
More information about the llvm-commits
mailing list