llvmbot wrote:
Failed to cherry-pick: 58323de2e5ed
https://github.com/llvm/llvm-project/actions/runs/8756326045
Please manually backport the fix and push it to your github fork. Once this is
done, please create a [pull
request](https://github.com/llvm/llvm-project/compare)
owenca wrote:
/cherry-pick 58323de2e5ed
https://github.com/llvm/llvm-project/pull/87953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
Ah, I still need to do that.
https://github.com/llvm/llvm-project/pull/87953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tstellar wrote:
@owenca Is there a PR for the cherry-pick ?
https://github.com/llvm/llvm-project/pull/87953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
@tstellar somehow this is not in 18.1.4, but the LLVM Release Status in the
Projects box says "Status: Done". Did I miss something here?
https://github.com/llvm/llvm-project/pull/87953
___
cfe-commits mailing list
https://github.com/owenca closed https://github.com/llvm/llvm-project/pull/87953
___
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/87953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mydeveloperday approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/87953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca milestoned
https://github.com/llvm/llvm-project/pull/87953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
The polly formatting failure should be ignored as CI is using clang-format
18.1.1 instead of the version built from this patch.
https://github.com/llvm/llvm-project/pull/87953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -538,16 +538,6 @@ void UnwrappedLineParser::calculateBraceTypes(bool
ExpectClassBody) {
if (Style.Language == FormatStyle::LK_Proto) {
ProbablyBracedList = NextTok->isOneOf(tok::comma, tok::r_square);
} else {
- // Skip NextTok over
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 943db678dadd6088629d08ec3e582bea0595f2d2
6d0c7e5602a227b1b7310be46553aa689e6a93e7 --
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Owen Pan (owenca)
Changes
Also fix unit tests and reformat polly.
Fixes #86550.
---
Full diff: https://github.com/llvm/llvm-project/pull/87953.diff
4 Files Affected:
- (modified) clang/lib/Format/UnwrappedLineParser.cpp
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/87953
Also fix unit tests and reformat polly.
Fixes #86550.
>From 6d0c7e5602a227b1b7310be46553aa689e6a93e7 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Sun, 7 Apr 2024 17:27:21 -0700
Subject: [PATCH] [clang-format]
14 matches
Mail list logo
