Author: hev
Date: 2024-07-21T09:30:48+08:00
New Revision: 3deacfb6a58ea3dcae27807209dffb227e9c786c
URL:
https://github.com/llvm/llvm-project/commit/3deacfb6a58ea3dcae27807209dffb227e9c786c
DIFF:
https://github.com/llvm/llvm-project/commit/3deacfb6a58ea3dcae27807209dffb227e9c786c.diff
LOG: Reve
Author: Vitaly Buka
Date: 2024-07-20T12:20:56-07:00
New Revision: 918587576c66f2640161c1b54fb128b0eb103880
URL:
https://github.com/llvm/llvm-project/commit/918587576c66f2640161c1b54fb128b0eb103880
DIFF:
https://github.com/llvm/llvm-project/commit/918587576c66f2640161c1b54fb128b0eb103880.diff
L
Author: Vitaly Buka
Date: 2024-07-20T12:19:14-07:00
New Revision: 1629eb6c05867047669f3b242f884182e4249d86
URL:
https://github.com/llvm/llvm-project/commit/1629eb6c05867047669f3b242f884182e4249d86
DIFF:
https://github.com/llvm/llvm-project/commit/1629eb6c05867047669f3b242f884182e4249d86.diff
L
@@ -41,8 +50,26 @@ void ImplementationInNamespaceCheck::check(
// Enforce that the namespace is the result of macro expansion
if (Result.SourceManager->isMacroBodyExpansion(NS->getLocation()) == false) {
-diag(NS->getLocation(), "the outermost namespace should be the '
@@ -41,8 +50,26 @@ void ImplementationInNamespaceCheck::check(
// Enforce that the namespace is the result of macro expansion
if (Result.SourceManager->isMacroBodyExpansion(NS->getLocation()) == false) {
-diag(NS->getLocation(), "the outermost namespace should be the '
5chmidti wrote:
Please add a test with an outer namespace that needs to be changed, but which
already has the hidden visibility attribute. That way, we'll be sure that the
replacement will replace the namespace name instead of breaking the attribute
into piec
@@ -41,8 +50,26 @@ void ImplementationInNamespaceCheck::check(
// Enforce that the namespace is the result of macro expansion
if (Result.SourceManager->isMacroBodyExpansion(NS->getLocation()) == false) {
-diag(NS->getLocation(), "the outermost namespace should be the '
https://github.com/5chmidti edited
https://github.com/llvm/llvm-project/pull/99681
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -51,7 +78,9 @@ void ImplementationInNamespaceCheck::check(
// instead.
if (NS->getVisibility() != Visibility::HiddenVisibility) {
diag(NS->getLocation(), "the '%0' macro should start with '%1'")
-<< RequiredNamespaceDeclMacroName << RequiredNamespaceDeclStart
https://github.com/5chmidti commented:
(not familiar with libc development)
One question may be if the outer namespace should be replaced with the libc
macro, or if the libc macro should be added around the other namespace. But
given that this check is for libc development, then the libc peopl
@@ -1,4 +1,6 @@
-// RUN: %check_clang_tidy %s llvmlibc-implementation-in-namespace %t
+// RUN: %check_clang_tidy %s llvmlibc-implementation-in-namespace %t -fix
5chmidti wrote:
No need to specify `-fix`, fixes are done automatically, e.g.,
https://github.com/llv
Author: Schrodinger ZHU Yifan
Date: 2024-07-20T08:11:31-07:00
New Revision: 4502f83ed026030e6886cdd3aac18e2f392d4877
URL:
https://github.com/llvm/llvm-project/commit/4502f83ed026030e6886cdd3aac18e2f392d4877
DIFF:
https://github.com/llvm/llvm-project/commit/4502f83ed026030e6886cdd3aac18e2f392d48
Author: Schrodinger ZHU Yifan
Date: 2024-07-20T08:09:24-07:00
New Revision: 52ea2963b4ae0de27816e7f1b3be45f4f021999a
URL:
https://github.com/llvm/llvm-project/commit/52ea2963b4ae0de27816e7f1b3be45f4f021999a
DIFF:
https://github.com/llvm/llvm-project/commit/52ea2963b4ae0de27816e7f1b3be45f4f02199
Author: Schrodinger ZHU Yifan
Date: 2024-07-20T08:04:24-07:00
New Revision: 9130500ce71f5ec38182ac8b397d8e6ab846782e
URL:
https://github.com/llvm/llvm-project/commit/9130500ce71f5ec38182ac8b397d8e6ab846782e
DIFF:
https://github.com/llvm/llvm-project/commit/9130500ce71f5ec38182ac8b397d8e6ab84678
https://github.com/arsenm updated
https://github.com/llvm/llvm-project/pull/99751
>From c10130fca09ad69a523969af1a5e24a11505fd5f Mon Sep 17 00:00:00 2001
From: Matt Arsenault
Date: Sat, 20 Jul 2024 00:01:47 +0400
Subject: [PATCH] AsmPrinter: Avoid use of MachineFunction::getMMI
---
llvm/lib/C
@@ -1,5 +1,5 @@
"Number","Name","Status","First released version"
mordante wrote:
This page has been shipped in several libc++ releases. I strongly like to have
it shipped in one release where P1614 is finished. I already have a TODO on my
private list to remo
https://github.com/s-barannikov approved this pull request.
https://github.com/llvm/llvm-project/pull/99751
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
llvmbot wrote:
@llvm/pr-subscribers-debuginfo
Author: Matt Arsenault (arsenm)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/99751.diff
2 Files Affected:
- (modified) llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp (+6-5)
- (modified) llvm/lib/CodeGen/AsmPrinter/DwarfCFI
https://github.com/arsenm ready_for_review
https://github.com/llvm/llvm-project/pull/99751
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
arsenm wrote:
> [!WARNING]
> This pull request is not mergeable via GitHub because a downstack PR is
> open. Once all requirements are satisfied, merge this PR as a stack href="https://app.graphite.dev/github/pr/llvm/llvm-project/99751?utm_source=stack-comment-downstack-mergeability-warning";
>
https://github.com/arsenm created
https://github.com/llvm/llvm-project/pull/99751
None
>From 31d06e474c36bdba7169e7ad68e48b32cfd34202 Mon Sep 17 00:00:00 2001
From: Matt Arsenault
Date: Sat, 20 Jul 2024 00:01:47 +0400
Subject: [PATCH] AsmPrinter: Avoid use of MachineFunction::getMMI
---
llvm
21 matches
Mail list logo
