https://github.com/cyndyishida approved this pull request.
https://github.com/llvm/llvm-project/pull/120149
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida approved this pull request.
https://github.com/llvm/llvm-project/pull/119908
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2539,6 +2550,18 @@ void DarwinClang::AddClangSystemIncludeArgs(const
llvm::opt::ArgList &DriverArgs
llvm::sys::path::append(P, "usr", "include");
addExternCSystemInclude(DriverArgs, CC1Args, P.str());
}
+
+ // Add default framework search paths
+ auto addFramew
@@ -150,3 +150,23 @@ clang::parseDarwinSDKInfo(llvm::vfs::FileSystem &VFS,
StringRef SDKRootPath) {
return llvm::make_error("invalid SDKSettings.json",
llvm::inconvertibleErrorCode());
}
+
+// For certain platforms/environments al
@@ -,8 +,15 @@ bool Parser::TryAnnotateTypeOrScopeTokenAfterScopeSpec(
}
}
- if (SS.isEmpty())
+ if (SS.isEmpty()) {
+if (getLangOpts().ObjC && !getLangOpts().CPlusPlus &&
cyndyishida wrote:
I'm wondering if it's valid for C code to enter
https://github.com/cyndyishida edited
https://github.com/llvm/llvm-project/pull/120149
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2539,6 +2550,18 @@ void DarwinClang::AddClangSystemIncludeArgs(const
llvm::opt::ArgList &DriverArgs
llvm::sys::path::append(P, "usr", "include");
addExternCSystemInclude(DriverArgs, CC1Args, P.str());
}
+
+ // Add default framework search paths
+ auto addFramew
https://github.com/cyndyishida edited
https://github.com/llvm/llvm-project/pull/120149
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,26 @@
+// UNSUPPORTED: system-windows
+// Windows is unsupported because we use the Unix path separator `/` in the
test.
+
+// Add default directories before running clang to check default
+// search paths.
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: cp -R %S/Inputs/M
https://github.com/cyndyishida commented:
Thank you for cleaning this up!!
https://github.com/llvm/llvm-project/pull/120149
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -339,13 +340,11 @@ void InitHeaderSearch::AddDefaultIncludePaths(
if (triple.isOSDarwin()) {
cyndyishida wrote:
I looked into this a bit more. IMO the only way to move all this code to the
driver is to either
1. changing search path order between Framew
https://github.com/cyndyishida approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/119473
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2,6 +2,7 @@ set(LLVM_LINK_COMPONENTS
OrcShared
OrcTargetProcess
Support
+ Core
cyndyishida wrote:
Why did you need to add link dependencies?
https://github.com/llvm/llvm-project/pull/119473
___
cfe-commi
cyndyishida wrote:
> Sorry, which env var are you referring to?
I'm suggesting an explicit test case that checks that `DEFAULT_SYSROOT` is set
and behaves as expected. It's ignored if sysroot or isysroot is passed, it's
used when they're missing.
https://github.com/llvm/llvm-project/pull/1
cyndyishida wrote:
> Current patch breaks test:
>
> ```
> // We pass --sysroot="" to defeat any -DDEFAULT_SYSROOT parameter.
> ```
>
> I think that just a pure hack in the test case, we can just update the test.
> @cyndyishida what do you think about the current approach?
SGTM too. I don't se
cyndyishida wrote:
> The main concern is it happens when setting `DEFAULT_SYSROOT`. If only
> `isysroot` is supported on macOS, does this mean `DEFAULT_SYSROOT` should be
> deprecated on Apple platforms?
I don't think it's deprecated in the sense we have plans to drop support for
it. The p
https://github.com/cyndyishida edited
https://github.com/llvm/llvm-project/pull/118543
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -339,13 +340,11 @@ void InitHeaderSearch::AddDefaultIncludePaths(
if (triple.isOSDarwin()) {
cyndyishida wrote:
Looks like that was previously attempted by @ldionne in
https://github.com/llvm/llvm-project/pull/75841
The problem was appending `/Library/F
@@ -346,6 +346,7 @@ void InitHeaderSearch::AddDefaultIncludePaths(
AddPath("/System/DriverKit/System/Library/Frameworks", System, true);
cyndyishida wrote:
Followup in: https://github.com/llvm/llvm-project/pull/118543
https://github.com/llvm/llvm-projec
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/118543
* Use the centralized API to add `SubFrameworks` for driverkit targets.
>From 764bf57ac39fddef74d45d16fffc14035bbbff72 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Tue, 3 Dec 2024 11:15:37 -0800
Subjec
cyndyishida wrote:
> > However, notably, even on Darwin, if you pass _both_ flags, you still get
> > the intended behavior of `-isysroot` affecting headers, and `--sysroot`
> > affecting libraries. Your change would break that.
>
> What's the use-case for this?
>
> Because for most users, you
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/115048
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida edited
https://github.com/llvm/llvm-project/pull/115048
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -346,6 +346,7 @@ void InitHeaderSearch::AddDefaultIncludePaths(
AddPath("/System/DriverKit/System/Library/Frameworks", System, true);
cyndyishida wrote:
I'd rather discuss DriverKit support separately, so I will merge this pr now.
> Also, why's there
cyndyishida wrote:
ping
https://github.com/llvm/llvm-project/pull/115048
___
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/115048
>From 9359df8624a75b7c0c361fdfcd9fc9102824dc68 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Tue, 5 Nov 2024 10:48:09 -0800
Subject: [PATCH 1/4] [Clang][Darwin] Introduce `SubFrameworks` as a SDK
defaul
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/115048
>From 9359df8624a75b7c0c361fdfcd9fc9102824dc68 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Tue, 5 Nov 2024 10:48:09 -0800
Subject: [PATCH 1/3] [Clang][Darwin] Introduce `SubFrameworks` as a SDK
defaul
https://github.com/cyndyishida edited
https://github.com/llvm/llvm-project/pull/115048
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -255,16 +255,21 @@ static void stubifyDirectory(const StringRef InputPath,
Context &Ctx) {
if (EC)
reportError(IT->path() + ": " + EC.message());
-// Skip header directories (include/Headers/PrivateHeaders) and module
cyndyishida wrote:
My s
@@ -347,6 +347,7 @@ void InitHeaderSearch::AddDefaultIncludePaths(
} else {
AddPath("/System/Library/Frameworks", System, true);
AddPath("/Library/Frameworks", System, true);
+AddPath("/System/Library/SubFrameworks", System, true);
cyndyishida wrote:
ping
https://github.com/llvm/llvm-project/pull/115048
___
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/115048
>From 9359df8624a75b7c0c361fdfcd9fc9102824dc68 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Tue, 5 Nov 2024 10:48:09 -0800
Subject: [PATCH 1/2] [Clang][Darwin] Introduce `SubFrameworks` as a SDK
defaul
cyndyishida wrote:
```
> git checkout -f baecb5c50287efebf2482739bfbe320c8147b7ff
error: unable to unlink old 'clang/lib/AST/Decl.cpp': Invalid argument
error: unable to unlink old 'clang/lib/Serialization/ASTReaderDecl.cpp':
Invalid argument
```
Seems like an unrelated issue is happening on the
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/115048
>From 9359df8624a75b7c0c361fdfcd9fc9102824dc68 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Tue, 5 Nov 2024 10:48:09 -0800
Subject: [PATCH 1/2] [Clang][Darwin] Introduce `SubFrameworks` as a SDK
defaul
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/115048
>From dd22f1347530d8f866822074ec9b89cb8ee56867 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Tue, 5 Nov 2024 10:48:09 -0800
Subject: [PATCH 1/2] [Clang][Darwin] Introduce `SubFrameworks` as a SDK
defaul
cyndyishida wrote:
> Do we also need to update `darwin::Linker::ConstructJob(...)`?
I thought about it, but clang already doesn't for today's default paths so I
decided against it. It seems only pass extra paths to the linker for
platform-specific directories like `DriverKit`. Xcode's `ld` pas
https://github.com/cyndyishida reopened
https://github.com/llvm/llvm-project/pull/115048
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cyndyishida wrote:
> `darwin::Linker::ConstructJob`
https://github.com/llvm/llvm-project/pull/115048
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/115048
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/115048
* Have clang always append & pass `System/Library/SubFrameworks` when
determining default sdk search paths.
* Teach `clang-installapi` to traverse there for framework input.
* Teach `llvm-readtapi` that the
https://github.com/cyndyishida approved this pull request.
I initially touched this code because I needed to be able to map back input
headers from a destination location (DSTROOT) to where they were resolved with
headermaps (SRCROOT) for `tapi`/`clang-installapi`. But looking at the code it
o
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/110724
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/110724
None
>From b78a9a6649adaa1b0f7252733995ece7d1cebee0 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Mon, 30 Sep 2024 17:07:06 -0700
Subject: [PATCH] [clang] Remove 'PCH' from more diagnostics that were wr
@@ -1637,31 +1637,42 @@ specifies availability for the current target platform,
the availability
attributes are ignored. Supported platforms are:
``ios``
+``ios_app_extension``
cyndyishida wrote:
I do think it's still useful to define what those platforms' v
@@ -1592,6 +1592,12 @@ succeeds but Clang emits a warning specifying that the
function is deprecated.
Finally, if Clang is instructed to compile code for macOS 10.7, the call
fails because ``f()`` is no longer available.
+Clang is instructed to compile code for a particular O
@@ -1592,6 +1592,12 @@ succeeds but Clang emits a warning specifying that the
function is deprecated.
Finally, if Clang is instructed to compile code for macOS 10.7, the call
fails because ``f()`` is no longer available.
+Clang is instructed to compile code for a particular O
https://github.com/cyndyishida approved this pull request.
https://github.com/llvm/llvm-project/pull/104653
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2953,7 +2953,15 @@ static bool sdkSupportsBuiltinModules(
case Darwin::MacOS:
return SDKVersion >= VersionTuple(15U);
cyndyishida wrote:
The extra test coverage is more for future-proofing. There is a decent amount
of `TargetVariantTriple` logic happ
@@ -1637,31 +1637,42 @@ specifies availability for the current target platform,
the availability
attributes are ignored. Supported platforms are:
``ios``
+``ios_app_extension``
Apple's iOS operating system. The minimum deployment target is specified
as part of the ``-ta
@@ -1637,31 +1637,42 @@ specifies availability for the current target platform,
the availability
attributes are ignored. Supported platforms are:
``ios``
+``ios_app_extension``
Apple's iOS operating system. The minimum deployment target is specified
as part of the ``-ta
@@ -1637,31 +1637,42 @@ specifies availability for the current target platform,
the availability
attributes are ignored. Supported platforms are:
``ios``
+``ios_app_extension``
cyndyishida wrote:
The description only describes the platform. Can you also incl
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
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
@@ -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
@@ -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
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
@@ -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
@@ -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
@@ -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
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
https://github.com/cyndyishida approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/102239
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/100636
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,300 @@
+//===- DirectoryScanner.cpp
---===//
+//
+// 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: Ap
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/100636
None
>From f96d9b79d7efb744182a5016295e6dd305564968 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Thu, 25 Jul 2024 12:15:45 -0700
Subject: [PATCH] [InstallAPI] Fix early return for scanning sub director
https://github.com/cyndyishida approved this pull request.
https://github.com/llvm/llvm-project/pull/100126
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/99520
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cyndyishida wrote:
> > I switched to the unit test, though I didn't figure out how to capture the
> > same sort of round-trip testing as with the lit test.
>
> Are you not seeing the same "infinite calls append the same token into
> CurDirTokens" issue via the unit test?
Oh yeah, that part is
cyndyishida wrote:
> This is similar feedback that I gave for #97654, could you test the change
> with a unit test in `DependencyDirectivesScannerTest.cpp` instead? It's much
> simpler to set up and execution-wise it's orders of magnitude more
> lightweight than a whole new lit test.
>
> For
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/99520
>From bcddefdce00a1e15f29181bc92eab86098a8b328 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Thu, 18 Jul 2024 08:56:24 -0700
Subject: [PATCH 1/2] [clang-scan-deps] Ignore import/include directives with
m
@@ -544,7 +544,7 @@ Scanner::lexIncludeFilename(const char *&First, const char
*const End) {
void Scanner::lexPPDirectiveBody(const char *&First, const char *const End) {
while (true) {
const dependency_directives_scan::Token &Tok = lexToken(First, End);
-if (Tok.is(
https://github.com/cyndyishida approved this pull request.
Thank you for looking into & resolving this again!
https://github.com/llvm/llvm-project/pull/99399
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
Author: Cyndy Ishida
Date: 2024-07-18T10:12:36-07:00
New Revision: 1c55586e9a475a09b7d769e7fc9a254e7150c972
URL:
https://github.com/llvm/llvm-project/commit/1c55586e9a475a09b7d769e7fc9a254e7150c972
DIFF:
https://github.com/llvm/llvm-project/commit/1c55586e9a475a09b7d769e7fc9a254e7150c972.diff
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/99520
Previously source input like `#import ` resulted in infinite calls append the
same token into `CurDirTokens`. This patch now ignores those directive lines if
they won't actually end up being compiled. (e.g.
cyndyishida wrote:
Thank you @zeroomega ! I am also running on arm64-mac and that test is passing.
What is your `cmake` invocation? I'd like to dig into the underlying cause of
non-stable ordering. It looks like the darwin-aarch64 is passing the test as
well https://lab.llvm.org/buildbot/#/bui
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/98171
___
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/98171
>From 03ee88e0b208d42a675dbbcf89ace869d193023b Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Tue, 9 Jul 2024 08:22:53 -0700
Subject: [PATCH] [InstallAPI] Don't look for linker directive symbols in
reexpo
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/98171
`$ld$previous` symbols need to be exported for them to be seen by clients. TAPI
cannot omit them in tbd files, so account for this in installapi verification
when handling reexport verification.
Reviewed in
@@ -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
@@ -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
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/94508
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cyndyishida wrote:
ping
https://github.com/llvm/llvm-project/pull/94508
___
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/94508
>From 95cc0c9a2b135706e80a5e73ef5e4257aa89984c Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Fri, 10 May 2024 09:19:22 -0700
Subject: [PATCH] [InstallAPI] Pick up input headers by directory traversal
Mat
@@ -97,6 +97,14 @@ class HeaderFile {
Other.Excluded, Other.Extra,
Other.Umbrella);
}
+
+ bool operator<(const HeaderFile &Other) const {
cyndyishida wrote:
I could infer, b
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/94508
>From c547d990aca29ecfe6f51d37c5c3f8712dfc5e44 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Fri, 10 May 2024 09:19:22 -0700
Subject: [PATCH 1/2] [InstallAPI] Pick up input headers by directory traversal
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
https://github.com/cyndyishida approved this pull request.
the InstallAPI changes LGTM, thanks!
https://github.com/llvm/llvm-project/pull/93190
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/91770
___
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/91770
>From 291412a203ea60465d4ecae9317f3490c59bfb50 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Thu, 2 May 2024 19:53:07 -0700
Subject: [PATCH 1/3] [InstallAPI] add JSON option to pass X arguments
---
.../
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/91770
>From 291412a203ea60465d4ecae9317f3490c59bfb50 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Thu, 2 May 2024 19:53:07 -0700
Subject: [PATCH 1/2] [InstallAPI] add JSON option to pass X arguments
---
.../
Author: Cyndy Ishida
Date: 2024-05-10T17:17:38-07:00
New Revision: 504cf554639360525c3f746e7296a242350b2af9
URL:
https://github.com/llvm/llvm-project/commit/504cf554639360525c3f746e7296a242350b2af9
DIFF:
https://github.com/llvm/llvm-project/commit/504cf554639360525c3f746e7296a242350b2af9.diff
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/91770
None
>From 480f1a6dd0c78c8018ac08259b4d3cba64c25165 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Thu, 2 May 2024 19:53:07 -0700
Subject: [PATCH] [InstallAPI] add JSON option to pass X arguments
---
..
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/90686
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cyndyishida wrote:
ping
https://github.com/llvm/llvm-project/pull/90686
___
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/90686
>From faabe7752dbb596f13e0e676b5c8db9fb8ad9258 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Thu, 18 Apr 2024 07:28:44 -0700
Subject: [PATCH] [InstallAPI] Support mutually exclusive parse options
Project
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/90686
>From c4fec501607059b1030b7c53794ad271194d Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Thu, 18 Apr 2024 07:28:44 -0700
Subject: [PATCH 1/2] [InstallAPI] Support mutually exclusive parse options
Pro
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/90686
Projects like libc use mutually exclusive macros to compile files multiple
times and then merge the result into the final library. For installapi to
accept these, we'd need to parse the same declarations in
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/90664
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida created
https://github.com/llvm/llvm-project/pull/90664
Add validation in the FileList reader to check that the headers exist and use
similar diagnostics in Options.cpp
>From 42ad6921fdf179647f6529ddeb7d060e93e0f52f Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Da
cyndyishida wrote:
> Overall I really like the direction of this. I'm curious if any of the
> maintainers from Apple have thoughts since they're the primary users of
> availability annotations.
While I like the approach of aligning availability parameters closer to
`llvm::Triple`, I am concer
cyndyishida wrote:
It appears your PR branch is based on a stale version of the target `llvm/main`
branch, resulting in more commits than what this PR intended for. Please merge
in the target branch and resolve any conflicts.
https://github.com/llvm/llvm-project/pull/89480
___
1 - 100 of 307 matches
Mail list logo
