evelez7 wrote:
* **#143919** https://app.graphite.dev/github/pr/llvm/llvm-project/143919?utm_source=stack-comment-icon";
target="_blank">https://static.graphite.dev/graphite-32x32-black.png"; alt="Graphite"
width="10px" height="10px"/> 👈 https://app.graphite.dev/github/pr/llvm/llvm-project/1439
https://github.com/evelez7 ready_for_review
https://github.com/llvm/llvm-project/pull/143919
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/143919
LLVM style prefers no default label on fully covered switches to warn if
new enums are added. This patch removes the default label for that
purpose or uses IT_default instead of default if that was the only enum
https://github.com/evelez7 edited
https://github.com/llvm/llvm-project/pull/143209
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,108 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --output=%t --format=json --executor=standalone %s
+// RUN: FileCheck %s < %t/GlobalNamespace/index.json
+
+class MyClass {};
+
+void myFunction(int Param);
+
+namespace NestedNamespace {
+} // namespace Nested
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From 741dce551684ee5818262ba3f4618ffcfa70a0d6 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/7] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
https://github.com/evelez7 edited
https://github.com/llvm/llvm-project/pull/142483
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,193 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --output=%t --format=json --executor=standalone %s
+// RUN: FileCheck %s < %t/GlobalNamespace/MyClass.json
+
+struct Foo;
+
+// This is a nice class.
+// It has some nice methods and fields.
+// @brief This is
https://github.com/evelez7 edited
https://github.com/llvm/llvm-project/pull/142483
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 edited
https://github.com/llvm/llvm-project/pull/142483
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,193 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --output=%t --format=json --executor=standalone %s
+// RUN: FileCheck %s < %t/GlobalNamespace/MyClass.json
+
+struct Foo;
+
+// This is a nice class.
+// It has some nice methods and fields.
+// @brief This is
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From 741dce551684ee5818262ba3f4618ffcfa70a0d6 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/7] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From 741dce551684ee5818262ba3f4618ffcfa70a0d6 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/7] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From 806c2c1f1846879f0217398f81f36a8e49447bdf Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/6] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
@@ -0,0 +1,194 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --output=%t --format=json --executor=standalone %s
+// RUN: FileCheck %s < %t/GlobalNamespace/MyClass.json
+
+struct Foo;
+
+// This is a nice class.
+// It has some nice methods and fields.
+// @brief This is
https://github.com/evelez7 ready_for_review
https://github.com/llvm/llvm-project/pull/142483
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From fa8b80f9bfe2b7faf765ed4cf60fb8cec30e1d48 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/5] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From fa8b80f9bfe2b7faf765ed4cf60fb8cec30e1d48 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/5] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From fa8b80f9bfe2b7faf765ed4cf60fb8cec30e1d48 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/4] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From fa8b80f9bfe2b7faf765ed4cf60fb8cec30e1d48 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/4] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From fa8b80f9bfe2b7faf765ed4cf60fb8cec30e1d48 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/3] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
evelez7 wrote:
> This sounds promising. I'm fine w/ adding a field to track this. BTW, what
> does clang do? I'm wondering if we should track more than 1-bit of info here.
As far as I can tell, inside the AST Clang makes use of the `isa<>` mechanisms
(which we could also leverage) or pointer u
evelez7 wrote:
If we're open to adding a flag in the base `Info` like `IsClassSpecialization`,
then we can probably easily deal with these by trying to reconstruct the
specialization's arguments (will be something like "Foo.json").
Functions don't produce their own files so function specializa
https://github.com/evelez7 closed
https://github.com/llvm/llvm-project/pull/142045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -198,37 +198,107 @@ static json::Value extractValue(const TypedefInfo &I) {
}
static json::Value extractValue(const CommentInfo &I) {
- assert((I.Kind == "BlockCommandComment" || I.Kind == "FullComment" ||
- I.Kind == "ParagraphComment" || I.Kind == "TextComment")
https://github.com/evelez7 approved this pull request.
https://github.com/llvm/llvm-project/pull/142665
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
evelez7 wrote:
> This is a good start, but I thin we'll want some more tests, and probably
> some unit test coverage. Unittesting is especially nice, since I believe this
> backend doesn't need any of the asset files, right?
Yeah, this doesn't need assets, so I'll be able to call things direct
@@ -198,37 +198,107 @@ static json::Value extractValue(const TypedefInfo &I) {
}
static json::Value extractValue(const CommentInfo &I) {
- assert((I.Kind == "BlockCommandComment" || I.Kind == "FullComment" ||
- I.Kind == "ParagraphComment" || I.Kind == "TextComment")
@@ -65,6 +65,35 @@ template <> struct ScalarEnumerationTraits {
}
};
+template <>
+struct llvm::yaml::ScalarEnumerationTraits {
evelez7 wrote:
This doesn't need the `llvm::yaml` qualification.
https://github.com/llvm/llvm-project/pull/142273
__
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From fa8b80f9bfe2b7faf765ed4cf60fb8cec30e1d48 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/2] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From cf68952c0cf4dfeda65c78c6d2326426bbf2d693 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-doc/
evelez7 wrote:
I think this patch is mostly ready in terms of functionality, but we should
decide if the emitted files should follow the HTML or YAML style of creation.
Right now, since I just ripped the Mustache generator code, it creates folders
for each namespace and emits the nested entiti
https://github.com/evelez7 edited
https://github.com/llvm/llvm-project/pull/142483
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
evelez7 wrote:
* **#142483** https://app.graphite.dev/github/pr/llvm/llvm-project/142483?utm_source=stack-comment-icon";
target="_blank">https://static.graphite.dev/graphite-32x32-black.png"; alt="Graphite"
width="10px" height="10px"/> 👈 https://app.graphite.dev/github/pr/llvm/llvm-project/1424
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/142483
None
>From 497637bbc1c5b20fb60357fbbfda55514a8681c2 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH] [clang-doc] add a JSON generator
---
clang-tools-extra/clan
https://github.com/evelez7 approved this pull request.
https://github.com/llvm/llvm-project/pull/142209
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 ready_for_review
https://github.com/llvm/llvm-project/pull/142045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142045
>From 7d94cff0737804c8d91712e7758439eb0cc07ff3 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Thu, 29 May 2025 14:44:29 -0700
Subject: [PATCH 1/4] [clang-doc] add tags to Mustache namespace template
namespace
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142045
>From 7d94cff0737804c8d91712e7758439eb0cc07ff3 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Thu, 29 May 2025 14:44:29 -0700
Subject: [PATCH 1/3] [clang-doc] add tags to Mustache namespace template
namespace
https://github.com/evelez7 edited
https://github.com/llvm/llvm-project/pull/142045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142045
>From 7d94cff0737804c8d91712e7758439eb0cc07ff3 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Thu, 29 May 2025 14:44:29 -0700
Subject: [PATCH 1/2] [clang-doc] add tags to Mustache namespace template
namespace
@@ -0,0 +1,74 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --format=mustache --output=%t --executor=standalone %s
+// RUN: FileCheck %s < %t/GlobalNamespace/index.html
+
+enum Color {
+ RED,
+ BLUE,
+ GREEN
+};
+
+class Foo;
+
+// CHECK:
+// CHECK-NEXT:
@@ -0,0 +1,74 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --format=mustache --output=%t --executor=standalone %s
+// RUN: FileCheck %s < %t/GlobalNamespace/index.html
+
+enum Color {
+ RED,
+ BLUE,
+ GREEN
+};
+
+class Foo;
+
+// CHECK:
+// CHECK-NEXT:
@@ -0,0 +1,13 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --format=mustache --output=%t --executor=standalone %s
+// RUN: FileCheck %s < %t/MyNamespace/index.html
+
+namespace MyNamespace {
+ class Foo;
+}
+
+// CHECK:
+// CHECK-NEXT:
+// CHECK-NEXT:
evelez7 wrote:
I cannibalized the tags from class-template.mustache. By happy accident, I
realized that there is an `extractScopeChildren` so enums can be generated,
which means the placeholders in `generateDocForInfo` (`OS << IT_enum`) can be
filled in if the logic is moved to their own funct
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/142045
namespace-template.mustache only rendered placeholder text. Enum and record
tags were added to the template. Now, we can render an index.html for the
global namespace and other namespaces.
Added tests and dele
https://github.com/evelez7 closed
https://github.com/llvm/llvm-project/pull/141976
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
evelez7 wrote:
CI passed besides the Linux test. Identical to what was discussed in #138065
https://github.com/llvm/llvm-project/pull/141976
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com
https://github.com/evelez7 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/141139
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
evelez7 wrote:
Sorry about that I was trying to get the PR changes via Github CLI...
https://github.com/llvm/llvm-project/pull/141139
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/141139
>From 7f346f476e89bc97c5d1780ef9f12246ebc4f86d Mon Sep 17 00:00:00 2001
From: Paul Kirth
Date: Wed, 21 May 2025 09:50:32 -0700
Subject: [PATCH] [clang-doc] Reenable time trace support
This patch re-enables -fti
@@ -13,7 +13,9 @@
#include "clang/Index/USRGeneration.h"
#include "llvm/ADT/StringExtras.h"
#include "llvm/ADT/StringSet.h"
+#include "llvm/Support/Error.h"
evelez7 wrote:
Error is already included via TimeProfiler.h
https://github.com/llvm/llvm-project/pull/
https://github.com/evelez7 unassigned
https://github.com/llvm/llvm-project/pull/141976
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/141976
The HTML FileCheck was missing the RUN command. Fixing that revealed an error
in the HTML check.
>From 38fa68d1e35cdffe4199e0cafde62edba54956ef Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Wed, 28 May 2025
https://github.com/evelez7 closed
https://github.com/llvm/llvm-project/pull/140856
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 approved this pull request.
https://github.com/llvm/llvm-project/pull/141269
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 closed
https://github.com/llvm/llvm-project/pull/141168
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/141168
None
>From 4c7294576a85fee8a373b407b39eeb0b35af4923 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Thu, 22 May 2025 17:31:37 -0700
Subject: [PATCH] [clang-doc] Precommit test for conversion function names
--
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/140856
>From b1a07c3dd19f2aa15abc286fdde0a2cb439dd7b7 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Thu, 22 May 2025 17:08:59 -0700
Subject: [PATCH] separate test from code change
---
clang-tools-extra/clang-doc/S
evelez7 wrote:
> would you mind pre-committing the test so its easy to see what's changing in
> the output with this patch?
Could you explain what this means please? Do you mean reorganizing the PR so
that the first commit is the test and then the second is the code change?
https://github.com
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/140856
>From e25581d28ecda89fe4e550da71e668b6ae749804 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Tue, 20 May 2025 23:26:02 -0700
Subject: [PATCH 1/2] [clang-doc] fix conversion names of dependent types
Fixes #59
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/140856
Fixes #59812
The names of conversion functions of template type parameters were being
emitted as "type-parameter-N-M". Now we check if the conversion type is a
TemplateTypeParmType and reconstruct the source n
evelez7 wrote:
Should note that #59812 also mentions the `requires` clause being omitted but
that should be addressed by a different issue/patch because that will be
applicable to all function types.
https://github.com/llvm/llvm-project/pull/140856
_
@@ -40,3 +51,4 @@ target_link_libraries(ClangDocTests
clangDoc
LLVMTestingSupport
)
+
evelez7 wrote:
extra newline
https://github.com/llvm/llvm-project/pull/138062
___
cfe-commits mailing list
cfe-commits@list
https://github.com/evelez7 approved this pull request.
https://github.com/llvm/llvm-project/pull/138062
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -6,4 +6,6 @@ set(LLVM_LINK_COMPONENTS
add_clang_library(clangDocSupport STATIC
File.cpp
- )
\ No newline at end of file
+ Utils.cpp
+ )
+
evelez7 wrote:
nit: extra newline
https://github.com/llvm/llvm-project/pull/138062
_
@@ -0,0 +1,61 @@
+//===--===//
+//
+// 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: Apac
@@ -74,7 +75,51 @@ static std::unique_ptr
NamespaceTemplate = nullptr;
static std::unique_ptr RecordTemplate = nullptr;
+static Error
+setupTemplate(std::unique_ptr &Template,
+ StringRef TemplatePath,
+ std::vector> Partials) {
+ auto T = Mustache
@@ -0,0 +1,56 @@
+// RUN: rm -rf %t
+// RUN: %clang_cc1 -extract-api --pretty-sgf
--emit-sgf-symbol-labels-for-testing \
+// RUN: --product-name=TypeAlias -triple arm64-apple-macosx -x c++-header %s
-o %t/type-alias.symbols.json -verify
+
+// RUN: FileCheck %s --input-file %t/
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/134007
>From 28879073b9d59dffa13f9523b4c9ec677b3a7b9c Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Tue, 1 Apr 2025 16:20:44 -0700
Subject: [PATCH 1/2] [clang][ExtractAPI] emit correct spelling for type
aliases
Pr
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/134007
Previously, C++11 type aliases were serialized using "typedef" regardless of
the source spelling.
This checks if the TypedefNameDecl is actually a TypeAliasDecl and corrects the
spelling.
>From 28879073b9d59df
https://github.com/evelez7 approved this pull request.
https://github.com/llvm/llvm-project/pull/110983
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 closed
https://github.com/llvm/llvm-project/pull/94674
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
evelez7 wrote:
Looks like this heavily affects libcxx, will a `DefaultIgnore` silence the
diagnostic?
https://github.com/llvm/llvm-project/pull/94789
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/94820
Change some parameters in DeclarationFragments.h to be passed by const
reference. Caught by cppcheck.
Fixes #92756 but doesn't address return value `RT` for `getTopLevelRecords`.
I'm not sure we'd want a const
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/94789
Deprecate the use of the template keyword before the qualified name of an alias
or class template without a template argument list. Introduced in
[P1787](https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2020/
evelez7 wrote:
I'm also preparing a patch for deprecating the use of template kw without an
arg list for alias and class templates. Decided to split that up since a lot of
tests require deprecation messages.
https://github.com/llvm/llvm-project/pull/94674
__
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/94674
Require a template argument list after a name prefixed by the template keyword
in nested name specifiers. Addresses [CWG
96](https://www.open-std.org/jtc1/sc22/wg21/docs/cwg_defects.html#96) which was
supersede
https://github.com/evelez7 closed
https://github.com/llvm/llvm-project/pull/80801
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/80801
>From 2c8c5129a9df16c87d9a89ce3864b857c438978e Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 5 Feb 2024 21:26:07 -0800
Subject: [PATCH 1/8] [clang] require template arg list after template kw
Require a t
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/80801
>From 2c8c5129a9df16c87d9a89ce3864b857c438978e Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 5 Feb 2024 21:26:07 -0800
Subject: [PATCH 1/7] [clang] require template arg list after template kw
Require a t
@@ -2995,13 +2996,23 @@ bool Parser::ParseUnqualifiedId(CXXScopeSpec &SS,
ParsedType ObjectType,
SS, ObjectType, ObjectHadErrors,
TemplateKWLoc ? *TemplateKWLoc : SourceLocation(), Id, IdLoc,
EnteringContext, Result, TemplateSpecified);
-else
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/80801
>From 2c8c5129a9df16c87d9a89ce3864b857c438978e Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 5 Feb 2024 21:26:07 -0800
Subject: [PATCH 1/6] [clang] require template arg list after template kw
Require a t
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/80801
>From 2c8c5129a9df16c87d9a89ce3864b857c438978e Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 5 Feb 2024 21:26:07 -0800
Subject: [PATCH 1/5] [clang] require template arg list after template kw
Require a t
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/80801
>From 2c8c5129a9df16c87d9a89ce3864b857c438978e Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 5 Feb 2024 21:26:07 -0800
Subject: [PATCH 1/4] [clang] require template arg list after template kw
Require a t
evelez7 wrote:
ping @Endilll
https://github.com/llvm/llvm-project/pull/80801
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/93205
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 closed
https://github.com/llvm/llvm-project/pull/77716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/77716
>From f0f25f2eb4f654c9a9f04c92deea9df2da6fc64c Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Wed, 10 Jan 2024 18:28:19 -0800
Subject: [PATCH] [clang][ExtractAPI] improve template argument name deduction
The n
@@ -1127,7 +1096,7 @@
DeclarationFragmentsBuilder::getFragmentsForVarTemplatePartialSpecialization(
.append("<", DeclarationFragments::FragmentKind::Text)
.append(getFragmentsForTemplateArguments(
Decl->getTemplateArgs().asArray(), Decl->getASTContext(),
evelez7 wrote:
ping
https://github.com/llvm/llvm-project/pull/80801
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/80801
>From fd07c0a15eb45c3e7eb400026ea7702ae909a11e Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 5 Feb 2024 21:26:07 -0800
Subject: [PATCH 1/4] [clang] require template arg list after template kw
Require a t
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/80801
>From fd07c0a15eb45c3e7eb400026ea7702ae909a11e Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 5 Feb 2024 21:26:07 -0800
Subject: [PATCH 1/3] [clang] require template arg list after template kw
Require a t
evelez7 wrote:
ping
https://github.com/llvm/llvm-project/pull/77716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
evelez7 wrote:
I was wondering why the Windows CI was failing but seems to be bugged according
to Discourse. Newest commit changes the error to a pedantic warning that
defaults to an error.
https://github.com/llvm/llvm-project/pull/80801
___
cfe-comm
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/80801
>From fd07c0a15eb45c3e7eb400026ea7702ae909a11e Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 5 Feb 2024 21:26:07 -0800
Subject: [PATCH 1/2] [clang] require template arg list after template kw
Require a t
@@ -868,6 +868,9 @@ def err_requires_expr_in_simple_requirement : Error<
"requires expression in requirement body; did "
"you intend to place it in a nested requirement? (add another 'requires' "
"before the expression)">;
+def err_missing_template_arg_list_after_template
https://github.com/evelez7 edited
https://github.com/llvm/llvm-project/pull/80801
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1414,7 +1414,7 @@ namespace dr96 { // dr96: no
// FIXME: This is ill-formed, because 'f' is not a template-id and does not
// name a class template.
// FIXME: What about alias templates?
evelez7 wrote:
The use of 'template' before an alias temp
https://github.com/evelez7 created
https://github.com/llvm/llvm-project/pull/80801
Require a template argument list after an identifier prefixed by the template
keyword. Introduced by [CWG
96](https://www.open-std.org/jtc1/sc22/wg21/docs/cwg_defects.html#96). Current
wording of [temp.names] i
1 - 100 of 124 matches
Mail list logo
