[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-19 Thread Martin Storsjö via cfe-commits
https://github.com/mstorsjo closed https://github.com/llvm/llvm-project/pull/109050 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-19 Thread Jan Svoboda via cfe-commits
https://github.com/jansvoboda11 approved this pull request. https://github.com/llvm/llvm-project/pull/109050 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-19 Thread Martin Storsjö via cfe-commits
https://github.com/mstorsjo updated https://github.com/llvm/llvm-project/pull/109050 From d7f46152baa1cffea16805bf7b522a237a54918f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Wed, 18 Sep 2024 00:03:00 +0300 Subject: [PATCH] [clang-scan-deps] Don't inspect Args[0] as a

[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-19 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff cb98fd977a25d3d778244da4012fd34acf7fab45 73dbd692945899708e2c127ba9972c20e5fcd30f --e

[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-19 Thread Martin Storsjö via cfe-commits
@@ -839,9 +839,11 @@ int clang_scan_deps_main(int argc, char **argv, const llvm::ToolContext &) { auto R = std::make_reverse_iterator(FlagsEnd); for (auto I = R, E = Args.rend(); I != E; ++I) { StringRef Arg = *I; +if ((I + 1) == E)

[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-19 Thread Martin Storsjö via cfe-commits
https://github.com/mstorsjo updated https://github.com/llvm/llvm-project/pull/109050 From 73dbd692945899708e2c127ba9972c20e5fcd30f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Wed, 18 Sep 2024 00:03:00 +0300 Subject: [PATCH] [clang-scan-deps] Don't inspect Args[0] as a

[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-18 Thread Jan Svoboda via cfe-commits
@@ -839,9 +839,11 @@ int clang_scan_deps_main(int argc, char **argv, const llvm::ToolContext &) { auto R = std::make_reverse_iterator(FlagsEnd); for (auto I = R, E = Args.rend(); I != E; ++I) { StringRef Arg = *I; +if ((I + 1) == E)

[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-18 Thread Rainer Orth via cfe-commits
rorth wrote: I can confirm that the patch fixes the error I'm seeing on the Solaris buildbots. Thanks. https://github.com/llvm/llvm-project/pull/109050 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/lis

[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-17 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Martin Storsjö (mstorsjo) Changes Since a26ec542371652e1d774696e90016fd5b0b1c191, we expand the executable name to an absolute path, if it isn't already one, if found in path. This broke a couple tests in some environments; when the clang

[clang] [clang-scan-deps] Don't inspect Args[0] as an option (PR #109050)

2024-09-17 Thread Martin Storsjö via cfe-commits
https://github.com/mstorsjo created https://github.com/llvm/llvm-project/pull/109050 Since a26ec542371652e1d774696e90016fd5b0b1c191, we expand the executable name to an absolute path, if it isn't already one, if found in path. This broke a couple tests in some environments; when the clang work