This revision was automatically updated to reflect the committed changes.
Closed by commit rG2a9a13d9cc5f: [OpenMP] Remove `-Bsymbolic` flag for device
linking in the GNU toolchain (authored by jhuber6).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136363/new/
https://reviews.llvm.org/D136363
Files:
clang/lib/Driver/ToolChains/Gnu.cpp
Index: clang/lib/Driver/ToolChains/Gnu.cpp
===================================================================
--- clang/lib/Driver/ToolChains/Gnu.cpp
+++ clang/lib/Driver/ToolChains/Gnu.cpp
@@ -586,13 +586,6 @@
CmdArgs.push_back("-lm");
}
- // If we are linking for the device all symbols should be bound locally. The
- // symbols are already protected which makes this redundant. This is only
- // necessary to work around a problem in bfd.
- // TODO: Remove this once 'lld' becomes the only linker for offloading.
- if (JA.isDeviceOffloading(Action::OFK_OpenMP))
- CmdArgs.push_back("-Bsymbolic");
-
// Silence warnings when linking C code with a C++ '-stdlib' argument.
Args.ClaimAllArgs(options::OPT_stdlib_EQ);
Index: clang/lib/Driver/ToolChains/Gnu.cpp
===================================================================
--- clang/lib/Driver/ToolChains/Gnu.cpp
+++ clang/lib/Driver/ToolChains/Gnu.cpp
@@ -586,13 +586,6 @@
CmdArgs.push_back("-lm");
}
- // If we are linking for the device all symbols should be bound locally. The
- // symbols are already protected which makes this redundant. This is only
- // necessary to work around a problem in bfd.
- // TODO: Remove this once 'lld' becomes the only linker for offloading.
- if (JA.isDeviceOffloading(Action::OFK_OpenMP))
- CmdArgs.push_back("-Bsymbolic");
-
// Silence warnings when linking C code with a C++ '-stdlib' argument.
Args.ClaimAllArgs(options::OPT_stdlib_EQ);
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
