[clang] Initialize SmallVector variable (PR #114434)

2024-10-31 Thread Arvind Sudarsanam via cfe-commits
asudarsa wrote: This is not the correct fix. Closing it. Thanks https://github.com/llvm/llvm-project/pull/114434 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Initialize SmallVector variable (PR #114434)

2024-10-31 Thread Arvind Sudarsanam via cfe-commits
https://github.com/asudarsa closed https://github.com/llvm/llvm-project/pull/114434 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Initialize SmallVector variable (PR #114434)

2024-10-31 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 commented: A default constructed vector should just be empty, I don't think it would cause the issue the sanitizer is seeing. https://github.com/llvm/llvm-project/pull/114434 ___ cfe-commits mailing list cfe-commits@lists.ll

[clang] Initialize SmallVector variable (PR #114434)

2024-10-31 Thread Youngsuk Kim via cfe-commits
JOE1994 wrote: Since `SmallVector DeviceLibFiles;` is default initialized to an empty vector, this change seems unnecessary. https://github.com/llvm/llvm-project/pull/114434 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.or

[clang] Initialize SmallVector variable (PR #114434)

2024-10-31 Thread via cfe-commits
llvmbot wrote: @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 diff --git a/clang/tool

[clang] Initialize SmallVector variable (PR #114434)

2024-10-31 Thread Arvind Sudarsanam via cfe-commits
https://github.com/asudarsa created https://github.com/llvm/llvm-project/pull/114434 None >From 9cfc10768adf19e41b22cd6a9fb2c781fdf4498e Mon Sep 17 00:00:00 2001 From: Arvind Sudarsanam Date: Thu, 31 Oct 2024 10:34:06 -0700 Subject: [PATCH] Initialize SmallVector variable Signed-off-by: Arvin