https://github.com/HighCommander4 approved this pull request.
Thanks! I have one suggestion for your consideration, otherwise this looks good
to me.
https://github.com/llvm/llvm-project/pull/68177
___
cfe-commits mailing list
cfe-commits@lists.llvm.or
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/68177
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -859,7 +872,9 @@ class InlayHintVisitor : public
RecursiveASTVisitor {
else
ForwardedParams = {Params.begin(), Params.end()};
-NameVec ParameterNames = chooseParameterNames(ForwardedParams);
+auto ForwardedParamsRef =
HighCommander4 wrote
https://github.com/MaskRay created
https://github.com/llvm/llvm-project/pull/69896
Fix #69889
>From 343f5abcd269701f526b64508237fb992ad7828d Mon Sep 17 00:00:00 2001
From: Fangrui Song
Date: Sun, 22 Oct 2023 22:04:31 -0700
Subject: [PATCH] -fstack-usage: fix filename for functions in an inclu
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Fangrui Song (MaskRay)
Changes
Fix #69889
---
Full diff: https://github.com/llvm/llvm-project/pull/69896.diff
2 Files Affected:
- (modified) clang/test/CodeGen/stack-usage.c (+8-5)
- (modified) llvm/lib/CodeGen/AsmPrinter/AsmPrinter.c
MaskRay added inline comments.
Herald added a subscriber: ormris.
Herald added a project: All.
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:1208
+
+ *StackUsageStream << MF.getFunction().getParent()->getName();
+ if (const DISubprogram *DSP = MF.getFunction().getSubpr
ChuanqiXu9 wrote:
@sam-mccall gentle ping~
https://github.com/llvm/llvm-project/pull/66462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/androm3da approved this pull request.
https://github.com/llvm/llvm-project/pull/69896
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Owen Pan
Date: 2023-10-22T23:01:27-07:00
New Revision: 0374bbba4c455e5f862a32581cc8d37690fb3b60
URL:
https://github.com/llvm/llvm-project/commit/0374bbba4c455e5f862a32581cc8d37690fb3b60
DIFF:
https://github.com/llvm/llvm-project/commit/0374bbba4c455e5f862a32581cc8d37690fb3b60.diff
LOG:
https://github.com/owenca closed https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: martinboehme
Date: 2023-10-23T08:18:12+02:00
New Revision: 3eed23d31caab9f19a8fa142aa26ee970a2c60ea
URL:
https://github.com/llvm/llvm-project/commit/3eed23d31caab9f19a8fa142aa26ee970a2c60ea
DIFF:
https://github.com/llvm/llvm-project/commit/3eed23d31caab9f19a8fa142aa26ee970a2c60ea.diff
https://github.com/martinboehme closed
https://github.com/llvm/llvm-project/pull/69601
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/69897
Fixes #69237.
>From 69e39f7e19e7f2538df0b8c6ba89d5efa493dd7b Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Sun, 22 Oct 2023 23:15:08 -0700
Subject: [PATCH] [clang-format] Don't alter SpacesRequiredBefore of final
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Owen Pan (owenca)
Changes
Fixes #69237.
---
Full diff: https://github.com/llvm/llvm-project/pull/69897.diff
2 Files Affected:
- (modified) clang/lib/Format/TokenAnnotator.cpp (+1-1)
- (modified) clang/unittests/Format/FormatTest
jh7370 wrote:
I can merge it for you, but first I just want to confirm that all 4 commits in
this PR are intended to be for the PR? If not, you'll need to rebase and force
push.
There's no need for you to manually squash the fixups (although you might as
well if you do end up rebasing). When
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/69897
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jevinskie created
https://github.com/llvm/llvm-project/pull/69899
Yowza, I haven't tried to upstream anything to LLVM in a long time. It was so
long ago I just had my coworker submit it for me with their SVN access...
Anyways, this adds `ObjCBridgeAttr`, `ObjCBridgeMutableAt
101 - 117 of 117 matches
Mail list logo