@@ -1908,7 +1908,7 @@ void clang::EmitClangDiagDocs(const RecordKeeper
&Records, raw_ostream &OS) {
for (const Record *G : DiagGroups) {
bool IsRemarkGroup = isRemarkGroup(G, DiagsInGroup);
auto &GroupInfo =
-DiagsInGroup[std::string(G->getValueAsString("Grou
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/119197
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/119197
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1908,7 +1908,7 @@ void clang::EmitClangDiagDocs(const RecordKeeper
&Records, raw_ostream &OS) {
for (const Record *G : DiagGroups) {
bool IsRemarkGroup = isRemarkGroup(G, DiagsInGroup);
auto &GroupInfo =
-DiagsInGroup[std::string(G->getValueAsString("Grou
https://github.com/jurahul approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/117479
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/116491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/116491
None
>From a69664043703506d4331ff008d97de0c978a8aed Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Fri, 15 Nov 2024 09:49:12 -0800
Subject: [PATCH] [NFC][Clang][TableGen] Fix file header comments
---
clang/
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/116491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/115959
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1419,16 +1412,14 @@ void clang::EmitClangDiagsDefs(const RecordKeeper
&Records, raw_ostream &OS,
InferPedantic inferPedantic(DGParentMap, Diags, DiagGroups, DiagsInGroup);
inferPedantic.compute(&DiagsInPedantic, (RecordVec*)nullptr);
- for (unsigned i = 0, e = Diags.
@@ -1419,16 +1412,14 @@ void clang::EmitClangDiagsDefs(const RecordKeeper
&Records, raw_ostream &OS,
InferPedantic inferPedantic(DGParentMap, Diags, DiagGroups, DiagsInGroup);
inferPedantic.compute(&DiagsInPedantic, (RecordVec*)nullptr);
- for (unsigned i = 0, e = Diags.
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/115959
>From d63b8a017e73586cc8c4c9d5afe4b0d59f81ea17 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Tue, 12 Nov 2024 15:38:02 -0800
Subject: [PATCH] [NFC][Clang] Use StringRef instead of string in
ClangDiagnosticEm
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/115212
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -145,7 +146,7 @@ static bool diagGroupBeforeByName(const Record *LHS, const
Record *RHS) {
RHS->getValueAsString("GroupName");
}
-using DiagsInGroupTy = std::map>;
+using DiagsInGroupTy = std::map>;
jurahul wrote:
Done.
https://github.com/llvm/l
@@ -117,15 +118,15 @@ namespace {
return CategoryIDs[CategoryString];
}
-typedef std::vector::const_iterator const_iterator;
+typedef std::vector::const_iterator const_iterator;
const_iterator begin() const { return CategoryStrings.begin(); }
const_
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/115962
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/115972
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/115959
>From aa016d306c717ddffc0a7227678d8928be9583b4 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Tue, 12 Nov 2024 15:38:02 -0800
Subject: [PATCH] [NFC][Clang] Use StringRef instead of string in
ClangDiagnosticEm
@@ -145,7 +146,7 @@ static bool diagGroupBeforeByName(const Record *LHS, const
Record *RHS) {
RHS->getValueAsString("GroupName");
}
-using DiagsInGroupTy = std::map>;
+using DiagsInGroupTy = std::map>;
jurahul wrote:
Will do, I see that you added th
@@ -158,21 +159,19 @@ static void groupDiagnostics(ArrayRef
Diags,
continue;
assert(R->getValueAsDef("Class")->getName() != "CLASS_NOTE" &&
"Note can't be in a DiagGroup");
-std::string GroupName =
-std::string(DI->getDef()->getValueAsString("G
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/115972
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/115962
>From 07c74bee58d1a56978b1cba00755cc3ca14cb71f Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Tue, 12 Nov 2024 15:51:54 -0800
Subject: [PATCH 1/2] [NFC][lang][TableGen] Simplify `EmitClangDiagsIndexName`
Simp
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/115962
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/115959
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/115972
Use StringRef and range for loops in Clang SACheckers and Syntax emitters.
>From 7e297e5641cead196772b33fc6633efe9837f09d Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Tue, 12 Nov 2024 17:06:08 -0800
Subject
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/115959
>From 5ba5485bbc8895743130f58779cf5c78eb0941aa Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Tue, 12 Nov 2024 15:38:02 -0800
Subject: [PATCH] [NFC][Clang] Use StringRef instead of string in
ClangDiagnosticEm
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/115959
>From 04b7434139ffdb412797e9d4fe7d31ecbdbe7748 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Tue, 12 Nov 2024 15:38:02 -0800
Subject: [PATCH] [NFC][Clang] Use StringRef instead of string in
ClangDiagnosticEm
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/115962
Simplify `EmitClangDiagsIndexName` to directly sort records instead of creating
an array of `RecordIndexElement` containing record name and sorting it.
>From 07c74bee58d1a56978b1cba00755cc3ca14cb71f Mon Sep 17
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/115959
>From cdcd09ec7c11a21da0804a6e886aff6410c42daf Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Tue, 12 Nov 2024 15:38:02 -0800
Subject: [PATCH] [NFC][Clang] Use StringRef instead of string in
ClangDiagnosticEm
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/115959
Use StringRef instead of std::string in ClangDiagnosticEmitter.
>From 5f67dd57f3e6333da438dc07a2efdfcb0c1874b1 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Tue, 12 Nov 2024 15:38:02 -0800
Subject: [PATCH] [
jurahul wrote:
Its not clear to me if the failure is real or intermittent:
FAILURE
Failed to match LIBC_NAMESPACE::mkdirat(AT_FDCWD, TEST_DIR, S_IRWXU) against
Succeeds(0).
Expected return value to be equal to 0 but got -1.
Expected errno to be equal to "Success" but got "File exists".
https:
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/115573
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1570,8 +1541,7 @@ static void emitDiagSubGroups(std::map &DiagsInGroup,
for (auto const &Group : GroupsInPedantic) {
const std::string &GroupName =
std::string(Group->getValueAsString("GroupName"));
- std::map::const_iterator RI =
-
@@ -1553,15 +1526,13 @@ static void emitDiagSubGroups(std::map &DiagsInGroup,
RecordVec &GroupsInPedantic, raw_ostream &OS) {
OS << "static const int16_t DiagSubGroups[] = {\n"
<< " /* Empty */ -1,\n";
- for (auto const &I : DiagsInGroup)
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/115573
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -970,10 +942,11 @@ struct DiagTextPrinter : DiagTextVisitor
{
void VisitPlural(PluralPiece *P) {
Result += "%plural{";
assert(P->Options.size() == P->OptionPrefixes.size());
-for (unsigned I = 0, End = P->Options.size(); I < End; ++I) {
- if (P->OptionPre
@@ -1553,15 +1526,13 @@ static void emitDiagSubGroups(std::map &DiagsInGroup,
RecordVec &GroupsInPedantic, raw_ostream &OS) {
OS << "static const int16_t DiagSubGroups[] = {\n"
<< " /* Empty */ -1,\n";
- for (auto const &I : DiagsInGroup)
@@ -1553,15 +1526,13 @@ static void emitDiagSubGroups(std::map &DiagsInGroup,
RecordVec &GroupsInPedantic, raw_ostream &OS) {
OS << "static const int16_t DiagSubGroups[] = {\n"
<< " /* Empty */ -1,\n";
- for (auto const &I : DiagsInGroup)
@@ -468,11 +442,9 @@ static StringRef getModifierName(ModifierType MT) {
return "objcclass";
case MT_ObjCInstance:
return "objcinstance";
- case MT_Unknown:
+ default:
jurahul wrote:
Done now.
https://github.com/llvm/llvm-project/pull/115573
_
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/115573
>From 635b24a4e057d85de6c2935ed817b93b34916ed3 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Fri, 8 Nov 2024 16:04:53 -0800
Subject: [PATCH] [NFC][Clang] Use range for loops in ClangDiagnosticsEmitter
Use ra
@@ -970,10 +942,11 @@ struct DiagTextPrinter : DiagTextVisitor
{
void VisitPlural(PluralPiece *P) {
Result += "%plural{";
assert(P->Options.size() == P->OptionPrefixes.size());
-for (unsigned I = 0, End = P->Options.size(); I < End; ++I) {
- if (P->OptionPre
@@ -468,11 +442,9 @@ static StringRef getModifierName(ModifierType MT) {
return "objcclass";
case MT_ObjCInstance:
return "objcinstance";
- case MT_Unknown:
+ default:
jurahul wrote:
Thanks. I'll revert this back.
https://github.com/llvm/llvm-proj
@@ -185,10 +174,8 @@ static void groupDiagnostics(ArrayRef
Diags,
}
// Assign unique ID numbers to the groups.
- unsigned IDNo = 0;
- for (std::map::iterator
- I = DiagsInGroup.begin(), E = DiagsInGroup.end(); I != E; ++I, ++IDNo)
-I->second.IDNo = IDNo;
+ fo
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/115573
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/115573
Use range based for loops in Clang diagnostics emitter.
>From af56060f8d27bb84ce642f6fd53de14ad8765bef Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Fri, 8 Nov 2024 16:04:53 -0800
Subject: [PATCH] [NFC][Clan
jurahul wrote:
> Is there any way you could split up into several patches? It's hard to follow
> when you do multiple things in one patch. Thanks!
Let me try to do it. May be a PR for just range based for loops to start with
https://github.com/llvm/llvm-project/pull/115212
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/115212
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/115212
>From 381cb66fbf539bb122a9970efca85fdce75f1b36 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Wed, 6 Nov 2024 11:22:55 -0800
Subject: [PATCH] [Clang] Refactor ClangDiagnosticEmitter to use more StringRef
Refa
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/115212
>From c3df877042c59ca9c9303de0f1707359cd91e9bd Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Wed, 6 Nov 2024 11:22:55 -0800
Subject: [PATCH] [Clang] Refactor ClangDiagnosticEmitter to use more StringRef
Refa
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/115212
Refactor ClangDiagnosticEmitter to use more StringRefs. Also use more range for
loops.
Verified that .inc files generated by clang build are identical with and
without the change.
>From b6c2a0777313c3f378c4bc
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/114331
>From 466b99a4d2f53daeecb4d0cfeb3ed5a05c2af03e Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Wed, 30 Oct 2024 16:51:36 -0700
Subject: [PATCH 1/2] [NFC] Remove referenced to deleted `ClangFormattedStatus`
fil
jurahul wrote:
> LGTM - thanks
Thanks. Will merge once checks are green
https://github.com/llvm/llvm-project/pull/114331
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/114331
None
>From 466b99a4d2f53daeecb4d0cfeb3ed5a05c2af03e Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Wed, 30 Oct 2024 16:51:36 -0700
Subject: [PATCH] [NFC] Remove referenced to deleted `ClangFormattedStatus`
f
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/114331
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/114331
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jurahul wrote:
Started https://github.com/llvm/llvm-project/pull/114331
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jurahul wrote:
Let me delete those references. The ones in generated_formatted_state.py are
fine
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/c
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jurahul wrote:
@AaronBallman go ahead with the merge since @owenca has approved?
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jurahul wrote:
Thanks. I haven't seen any responses from @mydeveloperday earlier, hopefully
@owenca can chime in.
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
jurahul wrote:
@AaronBallman can you PTAL?
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/113645
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/109220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/113645
>From 9192413d91525ef47cb7ee1837afcd0e89849481 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Thu, 24 Oct 2024 15:25:22 -0700
Subject: [PATCH] [Clang][TableGen] Use StringRef::str() instead of
std::string() c
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/113809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/113809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/113809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/113809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/113809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -49,40 +49,38 @@ using namespace llvm;
namespace {
class FlattenedSpelling {
- std::string V, N, NS;
+ StringRef V, N, NS;
jurahul wrote:
In short, I'll defer it to you. If we don't want to commit this, I am fine and
the find the reasoning behind if rea
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/113809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -49,40 +49,38 @@ using namespace llvm;
namespace {
class FlattenedSpelling {
- std::string V, N, NS;
+ StringRef V, N, NS;
jurahul wrote:
> TG is not perf critical
I have seen bugs about Tg perf, but may be not this particular backend,
https://github.
@@ -49,40 +49,38 @@ using namespace llvm;
namespace {
class FlattenedSpelling {
- std::string V, N, NS;
+ StringRef V, N, NS;
jurahul wrote:
In all these instances, the strings are derived from TableGen Records, so they
do outlive. I do agree that TG is no
jurahul wrote:
Thanks. I'll queue this behind https://github.com/llvm/llvm-project/pull/113809.
https://github.com/llvm/llvm-project/pull/113645
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
@@ -54,19 +54,18 @@ class FlattenedSpelling {
const Record &OriginalSpelling;
public:
- FlattenedSpelling(const std::string &Variety, const std::string &Name,
+ FlattenedSpelling(const std::string &Variety, StringRef Name,
const std::string &Namespace,
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/113809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/113809
>From 6b128746e5f6bfa13e0ea9a900f0bc8529437717 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Sun, 27 Oct 2024 06:51:55 -0700
Subject: [PATCH] [Clang][TableGen] Use StringRef in FlattenedSpelling
Change Flatt
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/113809
>From 60da18c88e63612c791ccbb423d752bac311650b Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Sun, 27 Oct 2024 06:51:55 -0700
Subject: [PATCH] [Clang][TableGen] Use StringRef in FlattenedSpelling
Change Flatt
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/113809
>From 5fd76fe7a8fe6a009966cf0d7d67a53b68b601a1 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Sun, 27 Oct 2024 06:51:55 -0700
Subject: [PATCH] [Clang][TableGen] Use StringRef in FlattenedSpelling
Change Flatt
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/113809
Change FlattenedSpelling to use StringRef instead of std::String.
>From 4a2e738c6b49ee7433cd9fa039b4028e862d92fe Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Sun, 27 Oct 2024 06:51:55 -0700
Subject: [PATCH]
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/113645
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/113645
Use `StringRef::str()` instead of std::string(StringRef) to cast from StringRef
to std::string.
>From c160176853c01ab33e53a508414d1532184dc6a8 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Thu, 24 Oct 2024
jurahul wrote:
> > > "[LLVM] Trim intrinsics" is not a helpful commit message. What exactly
> > > are you trying to achieve and why? Is there an RFC for this given it
> > > seems to be having significant consequences across the tree?
>
> >
>
> > Yes, please see here:
> > https://discourse.l
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/112791
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jurahul wrote:
> "[LLVM] Trim intrinsics" is not a helpful commit message. What exactly are
> you trying to achieve and why? Is there an RFC for this given it seems to be
> having significant consequences across the tree?
Yes, please see here:
https://discourse.llvm.org/t/rfc-compress-intrins
https://github.com/jurahul converted_to_draft
https://github.com/llvm/llvm-project/pull/112791
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/112791
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/112705
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -551,18 +552,18 @@ std::string BitsInit::getAsString() const {
// resolveReferences - If there are any field references that refer to fields
// that have been filled in, we can propagate the values now.
-Init *BitsInit::resolveReferences(Resolver &R) const {
+const Init *Bi
jurahul wrote:
Also, this enabled removing one const_cast<> in
https://github.com/llvm/llvm-project/pull/112261 (which I think is going to be
the final change for the time being to make more pointers const in TableGen
backends).
https://github.com/llvm/llvm-project/pull/112705
___
jurahul wrote:
> Do Init objects have any methods that can modify them? `const` was explicitly
> removed from some uses a long time ago
> [af8ee2c](https://github.com/llvm/llvm-project/commit/af8ee2cdeeb2ed35fdf0db8f7c06254c5d333ac4).
>
> Here's a very old thread about removing `const` from IR
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/112705
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/112705
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/112320
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/112318
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/112317
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1438,9 +1439,9 @@ void Intrinsic::emitReturn() {
emitNewLine();
}
-std::pair Intrinsic::DagEmitter::emitDag(DagInit *DI) {
+std::pair Intrinsic::DagEmitter::emitDag(const DagInit *DI)
{
// At this point we should only be seeing a def.
- DefInit *DefI = cast(DI->getO
1 - 100 of 414 matches
Mail list logo
