[clang] cac8116 - [OpenMP] Don't manually strip sections in the linker wrapper

Joseph Huber via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 15 17:35:37 PDT 2022


Author: Joseph Huber
Date: 2022-04-15T20:35:25-04:00
New Revision: cac81161ed1386b32443e13ab513c6a3d081d6a7

URL: https://github.com/llvm/llvm-project/commit/cac81161ed1386b32443e13ab513c6a3d081d6a7
DIFF: https://github.com/llvm/llvm-project/commit/cac81161ed1386b32443e13ab513c6a3d081d6a7.diff

LOG: [OpenMP] Don't manually strip sections in the linker wrapper

Summary:
The changes in D122987 ensures that the offloading sections always have
the SHF_EXCLUDE flag. This means that we do not need to manually strip
these sections for ELF or COFF targets.

Added: 
    

Modified: 
    clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp

Removed: 
    


################################################################################
diff  --git a/clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp b/clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp
index 2dc93ae0988e4..a11174b424203 100644
--- a/clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp
+++ b/clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp
@@ -65,7 +65,7 @@ static cl::OptionCategory
 static cl::opt<bool> StripSections(
     "strip-sections", cl::ZeroOrMore,
     cl::desc("Strip offloading sections from the host object file."),
-    cl::init(true), cl::cat(ClangLinkerWrapperCategory));
+    cl::init(false), cl::cat(ClangLinkerWrapperCategory));
 
 static cl::opt<std::string> LinkerUserPath("linker-path", cl::Required,
                                            cl::desc("Path of linker binary"),


        


More information about the cfe-commits mailing list