https://github.com/Meinersbur reopened
https://github.com/llvm/llvm-project/pull/89743
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/Meinersbur closed
https://github.com/llvm/llvm-project/pull/89743
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/Meinersbur updated
https://github.com/llvm/llvm-project/pull/89743
>From afadef1f53e03e2cf3a1695f3c693913b27382dd Mon Sep 17 00:00:00 2001
From: Michael Kruse
Date: Tue, 23 Apr 2024 13:03:11 +0200
Subject: [PATCH 1/3] [clang] Revise IDE folder structure
---
clang/CMakeLists
Meinersbur wrote:
> Oh the fun of PR numbers now lining up with Phabricator numbers. I thought
> you meant https://reviews.llvm.org/D89741 and was very confused until I saw
> #89741. :-D I'll take a look!
Well, seems I still have muscle memory from Phabricator times :-)
Thanks for the review!
https://github.com/AaronBallman approved this pull request.
I verified the layout seems reasonable in Visual Studio 2022; LGTM!
https://github.com/llvm/llvm-project/pull/89743
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
http
AaronBallman wrote:
> @AaronBallman Would you mind reviewing D89741 ? I was convinced a stakeholder
> there but they decided to drop out instead of approving it.
Oh the fun of PR numbers now lining up with Phabricator numbers. I thought you
meant https://reviews.llvm.org/D89741 and was very co
Meinersbur wrote:
@AaronBallman Would you mind reviewing D89741 ? I was convinced a stakeholder
there but they decided to drop out instead of approving it.
https://github.com/llvm/llvm-project/pull/89743
___
llvm-branch-commits mailing list
llvm-branc
AaronBallman wrote:
> @AaronBallman Thanks for having a look.
>
> > With this patch, I get errors when loading a visual studio solution
> > generated with these change, and all of clang's libraries are placed at the
> > top level. The error is a dialog box saying "The solution already contains
Meinersbur wrote:
@AaronBallman Thanks for having a look.
> With this patch, I get errors when loading a visual studio solution generated
> with these change, and all of clang's libraries are placed at the top level.
> The error is a dialog box saying "The solution already contains an item nam
https://github.com/AaronBallman commented:
With this patch, I get errors when loading a visual studio solution generated
with these change, and all of clang's libraries are placed at the top level.
The error is a dialog box saying "The solution already contains an item named
'clang'."
Another
Endilll wrote:
CC @AaronBallman
https://github.com/llvm/llvm-project/pull/89743
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/Meinersbur updated
https://github.com/llvm/llvm-project/pull/89743
>From afadef1f53e03e2cf3a1695f3c693913b27382dd Mon Sep 17 00:00:00 2001
From: Michael Kruse
Date: Tue, 23 Apr 2024 13:03:11 +0200
Subject: [PATCH 1/2] [clang] Revise IDE folder structure
---
clang/CMakeLists
llvmbot wrote:
@llvm/pr-subscribers-clang-analysis
Author: Michael Kruse (Meinersbur)
Changes
Reviewers of #89153 suggested to break up the patch into per-subproject
patches. This is the Clang part. See #89153 for the entire series and
motivation.
Update the folder titles for targets in
https://github.com/Meinersbur ready_for_review
https://github.com/llvm/llvm-project/pull/89743
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/Meinersbur edited
https://github.com/llvm/llvm-project/pull/89743
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/Meinersbur created
https://github.com/llvm/llvm-project/pull/89743
Reviewers of #89153 suggested to break up the patch into per-subproject
patches. This is the Clang part. See #89153 for the entire series and
motivation.
Update the folder titles for targets in the monorepos
16 matches
Mail list logo
