Initialize SmallVector variable

[asudarsa]

No description provided.

[llvmbot]

@llvm/pr-subscribers-clang

Author: Arvind Sudarsanam (asudarsa)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/114434.diff

1 Files Affected:

• (modified) clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp (+1-1)

diff --git a/clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp b/clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp
index 0639b95c76e218..fa37cbeb56316c 100644
--- a/clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp
+++ b/clang/tools/clang-sycl-linker/ClangSYCLLinker.cpp
@@ -237,7 +237,7 @@ Expected<StringRef> linkDeviceInputFiles(ArrayRef<std::string> InputFiles,
 // will be linked with input device files.
 // The list of files and its location are passed from driver.
 Expected<SmallVector<std::string>> getSYCLDeviceLibs(const ArgList &Args) {
-  SmallVector<std::string> DeviceLibFiles;
+  SmallVector<std::string> DeviceLibFiles{};
   StringRef LibraryPath;
   if (Arg *A = Args.getLastArg(OPT_library_path_EQ))
     LibraryPath = A->getValue();

[JOE1994]

Since SmallVector<std::string> DeviceLibFiles; is default initialized to an empty vector,
this change seems unnecessary.

[jhuber6]

A default constructed vector should just be empty, I don't think it would cause the issue the sanitizer is seeing.

[asudarsa]

This is not the correct fix. Closing it.

Thanks