https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/85100
* A lot of `tapi installapi` options are already shared with clang, but not
all. This patch handles installapi specific options by filtering for them in
initial argv input, then passing the rest to the clang
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Cyndy Ishida (cyndyishida)
Changes
* A lot of `tapi installapi` options are already shared with clang, but not
all. This patch handles installapi specific options by filtering for them in
initial argv input, then passing the rest to the c
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 e4edbae0aa6a9739954ee3b494b18f8c599d9d79
a71d8d338c9a5cc2f21957b7907f676f2f11d4d6 --
https://github.com/cyndyishida edited
https://github.com/llvm/llvm-project/pull/85100
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/85100
>From 7a7c5103a20b35a00c87fe47f1a4bc4a24dc205e Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Wed, 6 Mar 2024 09:16:20 -0800
Subject: [PATCH] [InstallAPI] Add installapi specific options & diagnostics
* A
@@ -126,9 +178,77 @@ bool Options::processFrontendOptions(InputArgList &Args) {
return true;
}
+std::vector
+Options::processAndFilterOutInstallAPIOptions(ArrayRef Args) {
+ std::unique_ptr Table;
+ Table.reset(createDriverOptTable());
+
+ unsigned MissingArgIndex, Missin
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/85100
>From 7a7c5103a20b35a00c87fe47f1a4bc4a24dc205e Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Wed, 6 Mar 2024 09:16:20 -0800
Subject: [PATCH 1/2] [InstallAPI] Add installapi specific options &
diagnostics
https://github.com/ributzka edited
https://github.com/llvm/llvm-project/pull/85100
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ributzka wrote:
Do you think this file is necessary?
https://github.com/llvm/llvm-project/pull/85100
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ributzka approved this pull request.
https://github.com/llvm/llvm-project/pull/85100
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cyndyishida wrote:
Strictly speaking no, I don't. From looking at the git history, I think this is
a pattern for supporting older includes through refactors. I figured to follow
it for parity with the other DiagnosticKind files.
https://github.com/llvm/llvm-
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/85100
>From 7a7c5103a20b35a00c87fe47f1a4bc4a24dc205e Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Wed, 6 Mar 2024 09:16:20 -0800
Subject: [PATCH 1/2] [InstallAPI] Add installapi specific options &
diagnostics
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/85100
>From 1281dfa43781cd544b80575d6b8e76a89204e7ff Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Wed, 6 Mar 2024 09:16:20 -0800
Subject: [PATCH] [InstallAPI] Add installapi specific options & diagnostics
* A
13 matches
Mail list logo