https://github.com/sstwcw closed
https://github.com/llvm/llvm-project/pull/148640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sstwcw updated
https://github.com/llvm/llvm-project/pull/148640
>From 59a2384a380ff329848e137b7b40a215981ac81d Mon Sep 17 00:00:00 2001
From: sstwcw
Date: Mon, 14 Jul 2025 14:32:40 +
Subject: [PATCH 1/2] Remove code related to trigraphs
When reviewing #147156, the review
https://github.com/sstwcw updated
https://github.com/llvm/llvm-project/pull/148640
>From 59a2384a380ff329848e137b7b40a215981ac81d Mon Sep 17 00:00:00 2001
From: sstwcw
Date: Mon, 14 Jul 2025 14:32:40 +
Subject: [PATCH 1/2] Remove code related to trigraphs
When reviewing #147156, the review
https://github.com/owenca edited
https://github.com/llvm/llvm-project/pull/148640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1300,22 +1293,16 @@ FormatToken *FormatTokenLexer::getNextToken() {
Style.TabWidth - (Style.TabWidth ? Column % Style.TabWidth : 0);
break;
case '\\':
- case '?':
- case '/':
-// The text was entirely whitespace when this loop was
https://github.com/mydeveloperday approved this pull request.
https://github.com/llvm/llvm-project/pull/148640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/148640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/naveen-seth commented:
Hi, I've requested the reviewers of
[#147156](https://github.com/llvm/llvm-project/pull/147156) for this since I
don't understand this topic all too well.
https://github.com/llvm/llvm-project/pull/148640
___
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: None (sstwcw)
Changes
When reviewing #147156, the reviewers pointed out that we didn't need
to support the trigraph. The code never handled it right.
In the debug build, this kind of input caused the assertion in the function
`co
https://github.com/sstwcw created
https://github.com/llvm/llvm-project/pull/148640
When reviewing #147156, the reviewers pointed out that we didn't need to
support the trigraph. The code never handled it right.
In the debug build, this kind of input caused the assertion in the function
`coun
10 matches
Mail list logo
