[clang] [InstallAPI] Hookup Input files & basic ASTVisitor (PR #82552)

2024-02-28 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/82552 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 6101cf3 - [InstallAPI] add missing link to clangAST

2024-02-28 Thread Cyndy Ishida via cfe-commits
Author: Cyndy Ishida Date: 2024-02-28T10:01:30-08:00 New Revision: 6101cf3164d77cfa4e2df75bf5262da9a2fb3d0a URL: https://github.com/llvm/llvm-project/commit/6101cf3164d77cfa4e2df75bf5262da9a2fb3d0a DIFF: https://github.com/llvm/llvm-project/commit/6101cf3164d77cfa4e2df75bf5262da9a2fb3d0a.diff

[clang] [InstallAPI] Hookup Input files & basic ASTVisitor (PR #82552)

2024-02-28 Thread Cyndy Ishida via cfe-commits
cyndyishida wrote: > Since clangFrontend depends on clangInstallAPI, Thanks for reporting this issue. Is it possible the issue is from a stale build? `clangFrontend` should no longer depend on `clangInstallAPI`. That dependency was removed in https://github.com/llvm/llvm-project/commit/0a518

[clang] 14faf0d - [TextAPI][InstallAPI] Fix documentation typos, NFC

2024-02-28 Thread Cyndy Ishida via cfe-commits
Author: Cyndy Ishida Date: 2024-02-28T14:24:42-08:00 New Revision: 14faf0d4aa841aafd0d1982391114bc35fda7e59 URL: https://github.com/llvm/llvm-project/commit/14faf0d4aa841aafd0d1982391114bc35fda7e59 DIFF: https://github.com/llvm/llvm-project/commit/14faf0d4aa841aafd0d1982391114bc35fda7e59.diff

[clang] [InstallAPI] Collect frontend attributes & ObjCInterface decls (PR #83378)

2024-02-28 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/83378 * This patch introduces a container class, for holding records and attributes only collectible from the clang frontend, which is a subclass of `llvm::MachO::RecordsSlice` * This also prunes out collecting de

[clang] [InstallAPI] Collect frontend attributes & ObjCInterface decls (PR #83378)

2024-02-29 Thread Cyndy Ishida via cfe-commits
@@ -16,6 +16,74 @@ using namespace llvm::MachO; namespace clang::installapi { +GlobalRecord *FrontendRecordsSlice::addGlobal( +StringRef Name, RecordLinkage Linkage, GlobalRecord::Kind GV, +const clang::AvailabilityInfo Avail, const Decl *D, const HeaderType Access,

[clang] [InstallAPI] Collect frontend attributes & ObjCInterface decls (PR #83378)

2024-02-29 Thread Cyndy Ishida via cfe-commits
@@ -0,0 +1,86 @@ +// RUN: rm -rf %t +// RUN: split-file %s %t +// RUN: sed -e "s|DSTROOT|%/t|g" %t/inputs.json.in > %t/inputs.json + +/// Check multiple targets are captured. cyndyishida wrote: Thanks for calling this out, forgot to remove. https://github.com/l

[clang] [InstallAPI] Collect frontend attributes & ObjCInterface decls (PR #83378)

2024-02-29 Thread Cyndy Ishida via cfe-commits
@@ -16,6 +16,74 @@ using namespace llvm::MachO; namespace clang::installapi { +GlobalRecord *FrontendRecordsSlice::addGlobal( +StringRef Name, RecordLinkage Linkage, GlobalRecord::Kind GV, +const clang::AvailabilityInfo Avail, const Decl *D, const HeaderType Access,

[clang] [InstallAPI] Collect frontend attributes & ObjCInterface decls (PR #83378)

2024-02-29 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/83378 >From 908e41485a148539b3d0699c55291f4f4e819858 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 28 Feb 2024 14:15:48 -0800 Subject: [PATCH 1/2] [InstallAPI] Collect frontend attributes during & ObjCInt

[clang] [InstallAPI] Use unique identifiers for input buffers (PR #83523)

2024-02-29 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/83523 None >From 82521efd33c253b34da149b5dfe6bbb7823589c1 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 29 Feb 2024 21:04:58 -0800 Subject: [PATCH] [InstallAPI] Use unique name for input buffers --- cla

[clang] [InstallAPI] Use unique identifiers for input buffers (PR #83523)

2024-02-29 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/83523 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Collect frontend attributes & ObjCInterface decls (PR #83378)

2024-03-01 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/83378 >From b3a2e850cbb7953902c6c64eb9c120422ec528bc Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 28 Feb 2024 14:15:48 -0800 Subject: [PATCH] [InstallAPI] Collect frontend attributes during & ObjCInterfa

[clang] [InstallAPI] Collect frontend attributes & ObjCInterface decls (PR #83378)

2024-03-01 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/83378 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] bcc6ca7 - [InstallAPI] Add missing link to clangLex

2024-03-01 Thread Cyndy Ishida via cfe-commits
Author: Cyndy Ishida Date: 2024-03-01T15:02:57-08:00 New Revision: bcc6ca7ff86518caa97c6f20735a4728e18caa78 URL: https://github.com/llvm/llvm-project/commit/bcc6ca7ff86518caa97c6f20735a4728e18caa78 DIFF: https://github.com/llvm/llvm-project/commit/bcc6ca7ff86518caa97c6f20735a4728e18caa78.diff

[clang] [llvm] [InstallAPI] Collect symbols from ObjC Ivars (PR #83632)

2024-03-01 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/83632 None >From 30f1fb6e8afbc0f793c3276ffda03fc50505ec73 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 29 Feb 2024 20:52:02 -0800 Subject: [PATCH] [InstallAPI] Collect symbols from ObjC Ivars --- clang

[clang] [llvm] [InstallAPI] Collect symbols from ObjC Ivars (PR #83632)

2024-03-01 Thread Cyndy Ishida via cfe-commits
@@ -27,12 +27,40 @@ __attribute__((objc_exception)) @interface Exception @end + +//--- Foo.framework/PrivateHeaders/Foo_Private.h +#import + +@interface ClassWithIvars : Visible { cyndyishida wrote: Something like ``` __attribute__((visibility("hidden"))

[clang] [llvm] [InstallAPI] Collect symbols from ObjC Ivars (PR #83632)

2024-03-04 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/83632 >From 30f1fb6e8afbc0f793c3276ffda03fc50505ec73 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 29 Feb 2024 20:52:02 -0800 Subject: [PATCH 1/2] [InstallAPI] Collect symbols from ObjC Ivars --- clang/i

[clang] [llvm] [InstallAPI] Collect symbols from ObjC Ivars (PR #83632)

2024-03-04 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/83632 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [InstallAPI] Collect global functions (PR #83952)

2024-03-04 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/83952 * Include whether functions are inlinable as they impact whether to add them into the tbd file and for future verification. * Fix how clang arguments got passed along, previously spacing was passed along to

[clang] [llvm] [InstallAPI] Collect global functions (PR #83952)

2024-03-05 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/83952 >From 580af4d537446637d44af03c147750af54eca05b Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Mon, 4 Mar 2024 10:46:18 -0800 Subject: [PATCH] [InstallAPI] Collect global functions * Include whether functi

[clang] ae9e1fd - [clang] Fix extractAPI typo in comments, NFC

2024-02-05 Thread Cyndy Ishida via cfe-commits
Author: Cyndy Ishida Date: 2024-02-05T14:14:21-08:00 New Revision: ae9e1fd2edb6eb11dfd6816d1e9013e39e21aa04 URL: https://github.com/llvm/llvm-project/commit/ae9e1fd2edb6eb11dfd6816d1e9013e39e21aa04 DIFF: https://github.com/llvm/llvm-project/commit/ae9e1fd2edb6eb11dfd6816d1e9013e39e21aa04.diff

[clang] [clang] Upstream XROS support in Clang (PR #78392)

2024-01-16 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/78392 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-13 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/81701 This patch adds support for expected InstallAPI inputs. InstallAPI accepts a well defined filelist of headers and how those headers represent a single library. InstallAPI captures header files to determine

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-14 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81701 >From 9d0da0010f145b23ce2b7e5b6183558609600789 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 13 Feb 2024 18:22:23 -0800 Subject: [PATCH] [clang][InstallAPI] Add input file support to library This pa

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-14 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81701 >From 9d0da0010f145b23ce2b7e5b6183558609600789 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 13 Feb 2024 18:22:23 -0800 Subject: [PATCH 1/2] [clang][InstallAPI] Add input file support to library Thi

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-14 Thread Cyndy Ishida via cfe-commits
@@ -0,0 +1,53 @@ +//===- InstallAPI/FileList.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: Apa

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-14 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81701 >From 9d0da0010f145b23ce2b7e5b6183558609600789 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 13 Feb 2024 18:22:23 -0800 Subject: [PATCH 1/3] [clang][InstallAPI] Add input file support to library Thi

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-14 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81701 >From 9d0da0010f145b23ce2b7e5b6183558609600789 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 13 Feb 2024 18:22:23 -0800 Subject: [PATCH 1/4] [clang][InstallAPI] Add input file support to library Thi

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-15 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81701 >From 9d0da0010f145b23ce2b7e5b6183558609600789 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 13 Feb 2024 18:22:23 -0800 Subject: [PATCH 1/5] [clang][InstallAPI] Add input file support to library Thi

[clang] [clang] Move `AvailabilityInfo` into AST library (PR #81897)

2024-02-15 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/81897 Previously this class was only used by ExtractAPI, but it will soon also be needed by InstallAPI. This patch should not change availability behavior but just centralizes the information next to what already

[clang] [clang] Move `AvailabilityInfo` into AST library (PR #81897)

2024-02-15 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81897 >From 2779a90bb248bd48840a17f72cdc68691a397577 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 15 Feb 2024 10:36:45 -0800 Subject: [PATCH] [clang] Move `AvailabilityInfo` into AST library Previously t

[clang] [clang][driver] Add missing integrated tool option in error message, NFC (PR #81926)

2024-02-15 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/81926 None >From 338373bcd0b4ec3a4d1275267d566ce8bedbd50a Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 15 Feb 2024 13:47:49 -0800 Subject: [PATCH] [clang][driver] Add missing integrated tool option in er

[clang] [clang][driver] Add missing integrated tool option in error message, NFC (PR #81926)

2024-02-15 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/81926 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Move `AvailabilityInfo` into AST library (PR #81897)

2024-02-15 Thread Cyndy Ishida via cfe-commits
@@ -57,6 +58,57 @@ class AvailabilitySpec { bool isOtherPlatformSpec() const { return Version.empty(); } }; +class Decl; + +/// Storage of availability attributes for a declaration. +struct AvailabilityInfo { + /// The domain is the platform for which this availability info

[clang] [clang] Move `AvailabilityInfo` into AST library (PR #81897)

2024-02-15 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/81897 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Set InstallAPI as a standalone tool instead of CC1 action (PR #82293)

2024-02-19 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/82293 Installapi has important distinctions when compared to the clang driver, so much that, it doesn't make much sense to try to integrate into it. This patch partially reverts the CC1 action & driver support to

[clang] [InstallAPI] Set InstallAPI as a standalone tool instead of CC1 action (PR #82293)

2024-02-19 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/82293 >From 7f6b664424f58782e8fcd085b252efa8e9415baa Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Mon, 19 Feb 2024 17:00:08 -0800 Subject: [PATCH] [InstallAPI] Set InstallAPI as a standalone tool instead of C

[clang] [InstallAPI] Set InstallAPI as a standalone tool instead of CC1 action (PR #82293)

2024-02-19 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/82293 >From 7f6b664424f58782e8fcd085b252efa8e9415baa Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Mon, 19 Feb 2024 17:00:08 -0800 Subject: [PATCH 1/2] [InstallAPI] Set InstallAPI as a standalone tool instead

[clang] [InstallAPI] Set InstallAPI as a standalone tool instead of CC1 action (PR #82293)

2024-02-19 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/82293 >From bddd3cc6f0989309599ebc68ce510706b32feba6 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Mon, 19 Feb 2024 17:00:08 -0800 Subject: [PATCH] [InstallAPI] Set InstallAPI as a standalone tool instead of C

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-19 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81701 >From 627a0869d11cc801c8cbfd4ef71c71d6d4b89ef4 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 13 Feb 2024 18:22:23 -0800 Subject: [PATCH] [clang][InstallAPI] Add input file support to library This pa

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-19 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81701 >From 4210939acc22027077e509c13a6004d00f752011 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 13 Feb 2024 18:22:23 -0800 Subject: [PATCH] [clang][InstallAPI] Add input file support to library This pa

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-20 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81701 >From 4210939acc22027077e509c13a6004d00f752011 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 13 Feb 2024 18:22:23 -0800 Subject: [PATCH 1/2] [clang][InstallAPI] Add input file support to library Thi

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-20 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/81701 >From 4210939acc22027077e509c13a6004d00f752011 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 13 Feb 2024 18:22:23 -0800 Subject: [PATCH 1/3] [clang][InstallAPI] Add input file support to library Thi

[clang] [clang][InstallAPI] Add input file support to library (PR #81701)

2024-02-20 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/81701 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Set InstallAPI as a standalone tool instead of CC1 action (PR #82293)

2024-02-20 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/82293 >From bd648b50002a64ec098ff450cc6dacc0bc6b Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 20 Feb 2024 10:06:30 -0800 Subject: [PATCH 1/2] Revert " [clang][InstallAPI] Introduce basic driver to wr

[clang] [InstallAPI] Set InstallAPI as a standalone tool instead of CC1 action (PR #82293)

2024-02-20 Thread Cyndy Ishida via cfe-commits
@@ -35,19 +36,19 @@ struct InstallAPIContext { /// Active target triple to parse. llvm::Triple TargetTriple{}; - /// Output stream to write TextAPI file to. - std::unique_ptr OS = nullptr; - - /// DiagnosticsEngine to report errors. - llvm::IntrusiveRefCntPtr Diags = n

[clang] [InstallAPI] Set InstallAPI as a standalone tool instead of CC1 action (PR #82293)

2024-02-20 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/82293 >From bd648b50002a64ec098ff450cc6dacc0bc6b Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 20 Feb 2024 10:06:30 -0800 Subject: [PATCH 1/3] Revert " [clang][InstallAPI] Introduce basic driver to wr

[clang] [InstallAPI] Set InstallAPI as a standalone tool instead of CC1 action (PR #82293)

2024-02-20 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/82293 >From bd648b50002a64ec098ff450cc6dacc0bc6b Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 20 Feb 2024 10:06:30 -0800 Subject: [PATCH 1/2] Revert " [clang][InstallAPI] Introduce basic driver to wr

[clang] [InstallAPI] Set InstallAPI as a standalone tool instead of CC1 action (PR #82293)

2024-02-21 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/82293 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 579ae44 - [InstallAPI] Add missing clangBasic library dependency

2024-02-21 Thread Cyndy Ishida via cfe-commits
Author: Cyndy Ishida Date: 2024-02-21T10:02:25-08:00 New Revision: 579ae446375b2bec6d329b612adfa0a74f7126e3 URL: https://github.com/llvm/llvm-project/commit/579ae446375b2bec6d329b612adfa0a74f7126e3 DIFF: https://github.com/llvm/llvm-project/commit/579ae446375b2bec6d329b612adfa0a74f7126e3.diff

[clang] e7bfe41 - [InstallAPI] Add additional missing library dependency

2024-02-21 Thread Cyndy Ishida via cfe-commits
Author: Cyndy Ishida Date: 2024-02-21T10:09:55-08:00 New Revision: e7bfe414a6abee31a8c83afbc8206e6249dd837d URL: https://github.com/llvm/llvm-project/commit/e7bfe414a6abee31a8c83afbc8206e6249dd837d DIFF: https://github.com/llvm/llvm-project/commit/e7bfe414a6abee31a8c83afbc8206e6249dd837d.diff

[clang] 5488e3e - [InstallAPI] add explicit dependency on llvmOption library

2024-02-21 Thread Cyndy Ishida via cfe-commits
Author: Cyndy Ishida Date: 2024-02-21T10:21:17-08:00 New Revision: 5488e3ea4869b32e82f7f7dfb3ba22cb7b5b9436 URL: https://github.com/llvm/llvm-project/commit/5488e3ea4869b32e82f7f7dfb3ba22cb7b5b9436 DIFF: https://github.com/llvm/llvm-project/commit/5488e3ea4869b32e82f7f7dfb3ba22cb7b5b9436.diff

[clang] [InstallAPI] Cleanup HeaderFile Interface & options handling, NFC (PR #82544)

2024-02-21 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/82544 None >From 1a5a7904364af0ab74b43f41fba819752074cd6f Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 21 Feb 2024 14:09:26 -0800 Subject: [PATCH] [InstallAPI] Cleanup HeaderFile Interface & options hand

[clang] [InstallAPI] Cleanup HeaderFile Interface & options handling, NFC (PR #82544)

2024-02-21 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/82544 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Hookup Input files & basic ASTVisitor (PR #82552)

2024-02-21 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/82552 This patch takes in json files as input to determine that header files to process, and in which order, to pass along for CC1 invocations. This patch also includes an ASTVisitor to collect simple global varia

[clang] [InstallAPI] Hookup Input files & basic ASTVisitor (PR #82552)

2024-02-21 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/82552 >From b85feababde1044be134bedc0a4eb9d80eb449a6 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 21 Feb 2024 14:56:02 -0800 Subject: [PATCH 1/2] [InstallAPI] Hookup Input files & basic ASTVisitor This p

[clang] [InstallAPI] Hookup Input files & basic ASTVisitor (PR #82552)

2024-02-21 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/82552 >From b85feababde1044be134bedc0a4eb9d80eb449a6 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 21 Feb 2024 14:56:02 -0800 Subject: [PATCH 1/3] [InstallAPI] Hookup Input files & basic ASTVisitor This p

[clang] bc841bb - [clang] Rename installapi tests, NFC

2024-02-22 Thread Cyndy Ishida via cfe-commits
Author: Cyndy Ishida Date: 2024-02-22T09:56:21-08:00 New Revision: bc841bb0f8b55d18ed97440df878d0121701a317 URL: https://github.com/llvm/llvm-project/commit/bc841bb0f8b55d18ed97440df878d0121701a317 DIFF: https://github.com/llvm/llvm-project/commit/bc841bb0f8b55d18ed97440df878d0121701a317.diff

[clang] [InstallAPI] Hookup Input files & basic ASTVisitor (PR #82552)

2024-02-22 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/82552 >From bb10dd04f1895b63d0183c1f1ee31d452233059b Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 21 Feb 2024 14:56:02 -0800 Subject: [PATCH] [InstallAPI] Hookup Input files & basic ASTVisitor This patch

[clang] [clang-installapi] ] Fix potential null pointer dereference in file enumeration (PR #97900)

2024-07-08 Thread Cyndy Ishida via cfe-commits
@@ -51,8 +51,14 @@ llvm::Expected enumerateFiles(FileManager &FM, StringRef Directory) { if (EC) return errorCodeToError(EC); +// Ensure the iterator is valid before dereferencing. +if (i == ie) cyndyishida wrote: Why does this make a diffe

[clang] [clang-installapi] ] Fix potential null pointer dereference in file enumeration (PR #97900)

2024-07-08 Thread Cyndy Ishida via cfe-commits
@@ -51,8 +51,14 @@ llvm::Expected enumerateFiles(FileManager &FM, StringRef Directory) { if (EC) return errorCodeToError(EC); +// Ensure the iterator is valid before dereferencing. +if (i == ie) + break; + // Skip files that do not exist. This usua

[clang] [clang][ExtractAPI] Compute inherited availability information (PR #103040)

2024-08-13 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida edited https://github.com/llvm/llvm-project/pull/103040 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][ExtractAPI] Compute inherited availability information (PR #103040)

2024-08-13 Thread Cyndy Ishida via cfe-commits
@@ -16,33 +16,101 @@ #include "clang/AST/Decl.h" #include "clang/Basic/TargetInfo.h" -namespace clang { +namespace { + +struct AvailabilitySet { cyndyishida wrote: Can you add a doc string or change the type name to match closer to what this captures? IIUC i

[clang] [clang][ExtractAPI] Compute inherited availability information (PR #103040)

2024-08-13 Thread Cyndy Ishida via cfe-commits
@@ -16,33 +16,101 @@ #include "clang/AST/Decl.h" #include "clang/Basic/TargetInfo.h" -namespace clang { +namespace { + +struct AvailabilitySet { + llvm::SmallVector Availabilities; + bool UnconditionallyDeprecated = false; + bool UnconditionallyUnavailable = false; -Avail

[clang] [clang][ExtractAPI] Compute inherited availability information (PR #103040)

2024-08-13 Thread Cyndy Ishida via cfe-commits
@@ -0,0 +1,149 @@ +// RUN: rm -rf %t +// RUN: %clang_cc1 -extract-api --pretty-sgf --emit-sgf-symbol-labels-for-testing -triple arm64-apple-macosx \ +// RUN: -x objective-c-header %s -o %t/output.symbols.json -verify + + +// RUN: FileCheck %s --input-file %t/output.symbols.json

[clang] [clang][ExtractAPI] Compute inherited availability information (PR #103040)

2024-08-13 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida commented: > Additionally this computes availability information for all platforms ahead Can you add a test for this? https://github.com/llvm/llvm-project/pull/103040 ___ cfe-commits mailing list cfe-commits@lists.llvm

[clang] [clang][ExtractAPI] Compute inherited availability information (PR #103040)

2024-08-13 Thread Cyndy Ishida via cfe-commits
@@ -16,33 +16,101 @@ #include "clang/AST/Decl.h" #include "clang/Basic/TargetInfo.h" -namespace clang { +namespace { + +struct AvailabilitySet { + llvm::SmallVector Availabilities; + bool UnconditionallyDeprecated = false; + bool UnconditionallyUnavailable = false; -Avail

[clang] [clang][ExtractAPI] Compute inherited availability information (PR #103040)

2024-08-13 Thread Cyndy Ishida via cfe-commits
@@ -16,33 +16,101 @@ #include "clang/AST/Decl.h" #include "clang/Basic/TargetInfo.h" -namespace clang { +namespace { + +struct AvailabilitySet { + llvm::SmallVector Availabilities; cyndyishida wrote: Can you mark this private, its not clear otherwise that on

[clang] [clang][ExtractAPI] Compute inherited availability information (PR #103040)

2024-08-13 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida edited https://github.com/llvm/llvm-project/pull/103040 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][ExtractAPI] Compute inherited availability information (PR #103040)

2024-08-14 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/103040 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Add test to validate cc1 arg forwading (PR #87666)

2024-04-04 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/87666 >From 6b2f262d1e69724d4eb96f5ad75aa2c202f3eb2d Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 4 Apr 2024 10:36:47 -0700 Subject: [PATCH] [InstallAPI] Add test to validate cc1 arg forwading --- .../F

[clang] [lldb] [llvm] [cmake] Build executables with -no_exported_symbols when building Apple toolchain (PR #87684)

2024-04-04 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/87684 >From 3ac6872328334384fa20998541fac841add767d9 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 4 Apr 2024 12:08:28 -0700 Subject: [PATCH 1/2] [cmake] Build executables with -no_exported_symbols when b

[clang] [lldb] [llvm] [cmake] Build executables with -no_exported_symbols when building Apple toolchain (PR #87684)

2024-04-04 Thread Cyndy Ishida via cfe-commits
@@ -673,6 +673,9 @@ option(LLVM_USE_OPROFILE option(LLVM_EXTERNALIZE_DEBUGINFO "Generate dSYM files and strip executables and libraries (Darwin Only)" OFF) +option(LLVM_ENABLE_NO_EXPORTED_SYMBOLS cyndyishida wrote: No preference here. Whatever seems easier

[clang] [lldb] [llvm] [cmake] Build executables with -no_exported_symbols when building Apple toolchain (PR #87684)

2024-04-04 Thread Cyndy Ishida via cfe-commits
@@ -654,6 +654,11 @@ enabled sub-projects. Nearly all of these variable names begin with Generate dSYM files and strip executables and libraries (Darwin Only). Defaults to OFF. +**LLVM_ENABLE_EXPORTED_SYMBOLS**:BOOL + When building executables, preserve symbol exports. D

[clang] [lldb] [llvm] [cmake] Build executables with -no_exported_symbols when building Apple toolchain (PR #87684)

2024-04-04 Thread Cyndy Ishida via cfe-commits
@@ -673,6 +673,9 @@ option(LLVM_USE_OPROFILE option(LLVM_EXTERNALIZE_DEBUGINFO "Generate dSYM files and strip executables and libraries (Darwin Only)" OFF) +option(LLVM_ENABLE_EXPORTED_SYMBOLS cyndyishida wrote: Would `LLVM_ENABLE_EXECUTABLES_WITH_EXPORTED_

[clang] [InstallAPI] Capture & compare load commands that may differ per arch slice (PR #87674)

2024-04-04 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/87674 >From a7e67582e35fe6c33a1e391eaf6cc7d5b49087c6 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 27 Mar 2024 12:17:01 -0400 Subject: [PATCH 1/2] [InstallAPI] Capture and compare load commands that may d

[clang] [InstallAPI] Capture & compare load commands that may differ per arch slice (PR #87674)

2024-04-04 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/87674 >From 7ef1a803c10cfef8f577a4e439221d778215464a Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 27 Mar 2024 12:17:01 -0400 Subject: [PATCH] [InstallAPI] Capture and compare load commands that may diffe

[clang] [InstallAPI] Add test to validate cc1 arg forwarding (PR #87666)

2024-04-04 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida edited https://github.com/llvm/llvm-project/pull/87666 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Add test to validate cc1 arg forwarding (PR #87666)

2024-04-04 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/87666 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Capture & compare load commands that may differ per arch slice (PR #87674)

2024-04-05 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/87674 >From 7ef1a803c10cfef8f577a4e439221d778215464a Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 27 Mar 2024 12:17:01 -0400 Subject: [PATCH 1/2] [InstallAPI] Capture and compare load commands that may d

[clang] [lldb] [llvm] [cmake] Build executables with -no_exported_symbols when building Apple toolchain (PR #87684)

2024-04-05 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/87684 >From 3ac6872328334384fa20998541fac841add767d9 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 4 Apr 2024 12:08:28 -0700 Subject: [PATCH 1/3] [cmake] Build executables with -no_exported_symbols when b

[clang] [InstallAPI] Capture & compare load commands that may differ per arch slice (PR #87674)

2024-04-05 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/87674 >From 7ef1a803c10cfef8f577a4e439221d778215464a Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Wed, 27 Mar 2024 12:17:01 -0400 Subject: [PATCH 1/3] [InstallAPI] Capture and compare load commands that may d

[clang] [lldb] [llvm] [cmake] Build executables with -no_exported_symbols when building Apple toolchain (PR #87684)

2024-04-05 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/87684 >From 3ac6872328334384fa20998541fac841add767d9 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 4 Apr 2024 12:08:28 -0700 Subject: [PATCH 1/4] [cmake] Build executables with -no_exported_symbols when b

[clang] [InstallAPI] Capture & compare load commands that may differ per arch slice (PR #87674)

2024-04-05 Thread Cyndy Ishida via cfe-commits
cyndyishida wrote: Thanks for reviewing @zixu-w ! https://github.com/llvm/llvm-project/pull/87674 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Capture & compare load commands that may differ per arch slice (PR #87674)

2024-04-05 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/87674 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [lldb] [llvm] [cmake] Build executables with -no_exported_symbols when building Apple toolchain (PR #87684)

2024-04-05 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/87684 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [lldb] [llvm] [cmake] Build executables with -no_exported_symbols when building Apple toolchain (PR #87684)

2024-04-05 Thread Cyndy Ishida via cfe-commits
@@ -1029,6 +1038,16 @@ macro(add_llvm_executable name) add_llvm_symbol_exports( ${name} ${LLVM_EXPORTED_SYMBOL_FILE} ) endif(LLVM_EXPORTED_SYMBOL_FILE) + if (NOT LLVM_ENABLE_EXPORTED_SYMBOLS_IN_EXECUTABLES) +if(LLVM_LINKER_SUPPORTS_NO_EXPORTED_SYMBOLS) + set_p

[clang] [lldb] [llvm] [cmake] Build executables with -no_exported_symbols when building Apple toolchain (PR #87684)

2024-04-05 Thread Cyndy Ishida via cfe-commits
cyndyishida wrote: > FYI, it looks like this change broke `compiler-rt` build, e.g. in > https://lab.llvm.org/buildbot/#/builders/270/builds/12485 Should be resolved by: https://github.com/llvm/llvm-project/commit/fe45029dbdee6b3df2dbeaed17c9dd598ec511f2 I suspect compiler-rt may be relying

[clang] [InstallAPI] Tie lifetime of FE objects to DylibVerifier (PR #88189)

2024-04-09 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/88189 A few verification checks need to happen until all AST's have been traversed, specifically for zippered framework checking. To keep source location until that time valid, hold onto to references of FrontendR

[clang] [InstallAPI] Tie lifetime of FE objects to DylibVerifier (PR #88189)

2024-04-09 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/88189 >From a7894f987b80f1916195c3ab15da5c33ab69ab00 Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Tue, 9 Apr 2024 13:04:18 -0700 Subject: [PATCH] [InstallAPI] Tie lifetime of FE objects to DylibVerifier A few

[clang] [InstallAPI] Tie lifetime of FE objects to DylibVerifier (PR #88189)

2024-04-09 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/88189 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Handle zippered frameworks (PR #88205)

2024-04-09 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/88205 A zippered framework is a single framework that can be loaded in both macOS and macatalyst processes. Broadly to InstallAPI, it means the same interface can represent two separate platforms. A dylib's symbo

[clang] [InstallAPI] Handle zippered frameworks (PR #88205)

2024-04-10 Thread Cyndy Ishida via cfe-commits
@@ -588,13 +622,58 @@ void DylibVerifier::visitSymbolInDylib(const Record &R, SymbolContext &SymCtx) { } } + const bool IsLinkerSymbol = SymbolName.starts_with("$ld$"); + + if (R.isVerified()) { +// Check for unavailable symbols. +// This should only occur in

[clang] [InstallAPI] Handle zippered frameworks (PR #88205)

2024-04-10 Thread Cyndy Ishida via cfe-commits
@@ -588,13 +622,58 @@ void DylibVerifier::visitSymbolInDylib(const Record &R, SymbolContext &SymCtx) { } } + const bool IsLinkerSymbol = SymbolName.starts_with("$ld$"); + + if (R.isVerified()) { +// Check for unavailable symbols. +// This should only occur in

[clang] [clang][docs] Modernize attribute docs for darwin specifics (PR #88448)

2024-04-11 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/88448 * Generally recommend target triples. But replace `m*version-min` with `mtargetos`. * Also include test coverage for -mtargetos=visionos >From 0039cbc0d53a4ff8530b78657f8365229162516b Mon Sep 17 00:00:00 200

[clang] [clang][docs] Modernize attribute docs for darwin specifics (PR #88448)

2024-04-11 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida updated https://github.com/llvm/llvm-project/pull/88448 >From 0039cbc0d53a4ff8530b78657f8365229162516b Mon Sep 17 00:00:00 2001 From: Cyndy Ishida Date: Thu, 11 Apr 2024 14:10:15 -0700 Subject: [PATCH 1/2] [clang][docs] Modernize attribute docs for darwin specifi

[clang] [clang][docs] Modernize attribute docs for darwin specifics (PR #88448)

2024-04-11 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/88448 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [InstallAPI] Handle zippered frameworks (PR #88205)

2024-04-11 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida closed https://github.com/llvm/llvm-project/pull/88205 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 79dca25 - [InstallAPI] Replace std::string -> StringRef

2024-04-12 Thread Cyndy Ishida via cfe-commits
Author: Cyndy Ishida Date: 2024-04-12T22:05:12-07:00 New Revision: 79dca25f4a0fefd47e9e37d9ce47d84dc0b3bedb URL: https://github.com/llvm/llvm-project/commit/79dca25f4a0fefd47e9e37d9ce47d84dc0b3bedb DIFF: https://github.com/llvm/llvm-project/commit/79dca25f4a0fefd47e9e37d9ce47d84dc0b3bedb.diff

[clang] [InstallAPI] Pick up input headers by directory traversal (PR #94508)

2024-06-05 Thread Cyndy Ishida via cfe-commits
https://github.com/cyndyishida created https://github.com/llvm/llvm-project/pull/94508 Match TAPI behavior and allow input headers to be resolved via a passed directory, which is expected to be a library sitting in a build directory. >From c547d990aca29ecfe6f51d37c5c3f8712dfc5e44 Mon Sep 17 00

<    1   2   3   4   >