[clang] 83ce977 - [clang-sycl-linker] Fix use of uninitialized memory in temp files (#114488)

via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 1 05:20:06 PDT 2024


Author: Nick Sarnie
Date: 2024-11-01T07:20:02-05:00
New Revision: 83ce977b366a448609750630a6c1a23146c780d2

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

LOG: [clang-sycl-linker] Fix use of uninitialized memory in temp files (#114488)

This fixes the current sanitizer CI
[failures](https://lab.llvm.org/buildbot/#/builders/169/builds/4839/steps/13/logs/stdio).
I manually confirmed the fix with a MemorySanitizer build.

Signed-off-by: Sarnie, Nick <nick.sarnie at intel.com>

Added: 
    

Modified: 
    clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp

Removed: 
    


################################################################################
diff  --git a/clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp b/clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp
index 0639b95c76e218..076458a275d986 100644
--- a/clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp
+++ b/clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp
@@ -230,7 +230,7 @@ Expected<StringRef> linkDeviceInputFiles(ArrayRef<std::string> InputFiles,
   CmdArgs.push_back("--suppress-warnings");
   if (Error Err = executeCommands(*LLVMLinkPath, CmdArgs))
     return std::move(Err);
-  return *OutFileOrErr;
+  return Args.MakeArgString(*OutFileOrErr);
 }
 
 // This utility function is used to gather all SYCL device library files that


        


More information about the cfe-commits mailing list