[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/81571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

https://github.com/ributzka approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/81571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81571 >From 0f938ae8480f74a44bf6fe4aac6105e457315bb9 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Mon, 12 Feb 2024 20:53:25 -0800 Subject: [PATCH 1/2] [clang][InstallAPI] Introduce basic driver to write out

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

@@ -4319,6 +4324,12 @@ void Driver::BuildActions(Compilation , DerivedArgList , if (!MergerInputs.empty()) Actions.push_back( C.MakeAction(MergerInputs, types::TY_Image)); + } else if (Args.hasArg(options::OPT_installapi)) { +assert(Inputs.size() == 1

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

@@ -4319,6 +4324,12 @@ void Driver::BuildActions(Compilation , DerivedArgList , if (!MergerInputs.empty()) Actions.push_back( C.MakeAction(MergerInputs, types::TY_Image)); + } else if (Args.hasArg(options::OPT_installapi)) { +assert(Inputs.size() == 1

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

@@ -0,0 +1,65 @@ +//===- InstallAPI/Context.h -*- C++ -*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier:

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

@@ -94,6 +94,7 @@ TYPE("lto-bc", LTO_BC, INVALID, "o", phases TYPE("ast", AST, INVALID, "ast", phases::Compile, phases::Backend, phases::Assemble, phases::Link) TYPE("ifs", IFS,

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

@@ -4319,6 +4324,12 @@ void Driver::BuildActions(Compilation , DerivedArgList , if (!MergerInputs.empty()) Actions.push_back( C.MakeAction(MergerInputs, types::TY_Image)); + } else if (Args.hasArg(options::OPT_installapi)) { +assert(Inputs.size() == 1

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

@@ -804,4 +804,7 @@ def warn_android_unversioned_fallback : Warning< def err_drv_triple_version_invalid : Error< "version '%0' in target triple '%1' is invalid">; + +def err_drv_installapi_unsupported : Error< + "the clang compiler does not support '%0' for InstallAPI">;

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

https://github.com/ributzka edited https://github.com/llvm/llvm-project/pull/81571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

@@ -45,6 +45,7 @@ ".rs", ".ifs", ".rc", +".tbd", ributzka wrote: This doesn't seem to be used by the current tests. https://github.com/llvm/llvm-project/pull/81571 ___ cfe-commits mailing list

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

@@ -0,0 +1,65 @@ +//===- InstallAPI/Context.h -*- C++ -*-===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier:

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

https://github.com/ributzka commented: It is great to finally see this come together :D Thank you Cyndy https://github.com/llvm/llvm-project/pull/81571 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

cyndyishida wrote: In case anyone is interested, Here are relevant links for upstreaming `tapi` in LLVM: * [Inital RFC](https://discourse.llvm.org/t/rfc-open-sourcing-and-contributing-tapi-back-to-the-llvm-community/46214/24) * [Continued

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

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 4af24d4ab76539706bfbceec4b3923426fb1b9e7 0f938ae8480f74a44bf6fe4aac6105e457315bb9 --

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

llvmbot wrote: @llvm/pr-subscribers-clang-driver Author: Cyndy Ishida (cyndyishida) Changes This introduces a basic outline of installapi as a clang driver option. It captures relevant information as cc1 args, which are common arguments already passed to the linker to encode into TBD

[clang] [clang][InstallAPI] Introduce basic driver to write out tbd files (PR #81571)

https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/81571 This introduces a basic outline of installapi as a clang driver option. It captures relevant information as cc1 args, which are common arguments already passed to the linker to encode into TBD file outputs.