kpdev wrote:
> LGTM for the LoongArch change. We'll see if @kpdev has any objections later.
LGTM also. Thank you for the patch
https://github.com/llvm/llvm-project/pull/127555
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
kpdev wrote:
@phuang @nico Reverted. I will reland it after investigating the issue with GN
build bot
https://github.com/llvm/llvm-project/pull/118192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
Author: Pavel Kosov
Date: 2024-12-14T14:46:47+03:00
New Revision: 1911919682c863643787b30286bb67359c7932f4
URL:
https://github.com/llvm/llvm-project/commit/1911919682c863643787b30286bb67359c7932f4
DIFF:
https://github.com/llvm/llvm-project/commit/1911919682c863643787b30286bb67359c7932f4.diff
L
kpdev wrote:
@nico Should I revert this patch for now?
https://github.com/llvm/llvm-project/pull/118192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kpdev wrote:
Hm... No, it should work. @phuang May I kindly ask you to take a look at this
issue? http://45.33.8.238/linux/155432/step_6.txt
https://github.com/llvm/llvm-project/pull/118192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
kpdev wrote:
Agree. Tests would be useful here. Thanks for mentioning it
https://github.com/llvm/llvm-project/pull/118192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kpdev closed https://github.com/llvm/llvm-project/pull/118192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kpdev wrote:
LGTM
https://github.com/llvm/llvm-project/pull/118192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kpdev wrote:
@phuang Hi! Thank you for the patch. Will take a look in the nearest pair of
days
https://github.com/llvm/llvm-project/pull/118192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
kpdev wrote:
> LGTM, do you need someone to land this on your behalf?
Thank you, Aaron. Commit landed
https://github.com/llvm/llvm-project/pull/78253
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
https://github.com/kpdev closed https://github.com/llvm/llvm-project/pull/78253
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kpdev updated https://github.com/llvm/llvm-project/pull/78253
>From ebe0b998fc0476aed38fff7fcf368a08ddbda8a3 Mon Sep 17 00:00:00 2001
From: Nikolai Kholiavin
Date: Wed, 20 Dec 2023 16:19:03 +
Subject: [PATCH] [cfi][CodeGen] Call SetLLVMFunctionAttributes{,ForDefinition}
o
kpdev wrote:
> Should we add a release note to `clang/docs/ReleaseNotes.rst` so users know
> about the changes?
Seems reasonable ) Should I add it somewhere to
[`Sanitizers`](https://github.com/llvm/llvm-project/blob/main/clang/docs/ReleaseNotes.rst#sanitizers)
?
https://github.com/llvm/llv
@@ -3443,11 +3443,28 @@ void CodeGenFunction::EmitCfiSlowPathCheck(
void CodeGenFunction::EmitCfiCheckStub() {
llvm::Module *M = &CGM.getModule();
auto &Ctx = M->getContext();
+ auto &C = getContext();
kpdev wrote:
Changed, please review
https://github.c
https://github.com/kpdev updated https://github.com/llvm/llvm-project/pull/78253
>From 9850957ff41dd5cbe72fd76925a32ad9c277d5e1 Mon Sep 17 00:00:00 2001
From: Nikolai Kholiavin
Date: Wed, 20 Dec 2023 16:19:03 +
Subject: [PATCH] [cfi][CodeGen] Call SetLLVMFunctionAttributes{,ForDefinition}
o
kpdev wrote:
@AaronBallman @kongy Hi! Could you please take a look at this patch?
https://github.com/llvm/llvm-project/pull/78253
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kpdev wrote:
Ping
https://github.com/llvm/llvm-project/pull/78253
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kpdev wrote:
The same thing was done for `__cfi_check_fail` in the following patch:
https://reviews.llvm.org/D70692
https://github.com/llvm/llvm-project/pull/78253
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/
https://github.com/kpdev created https://github.com/llvm/llvm-project/pull/78253
…i_check
This causes __cfi_check, just as __cfi_check_fail, to get the proper
target-specific attributes, in particular uwtable for unwind table generation.
Previously, nounwind attribute could be inferred for __c
https://github.com/kpdev closed https://github.com/llvm/llvm-project/pull/68656
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kpdev wrote:
@brad0 Thank you for the patch, and sorry for late reply - there were wrong
settings with notifications, won't happen again
https://github.com/llvm/llvm-project/pull/68656
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
Author: Pavel Kosov
Date: 2023-03-21T17:26:36+03:00
New Revision: 88e5206f2c96a34e23a4d63f0a38afb2db044f0a
URL:
https://github.com/llvm/llvm-project/commit/88e5206f2c96a34e23a4d63f0a38afb2db044f0a
DIFF:
https://github.com/llvm/llvm-project/commit/88e5206f2c96a34e23a4d63f0a38afb2db044f0a.diff
L
Author: Pavel Kosov
Date: 2023-03-20T18:20:41+03:00
New Revision: fa326c21919faea33b0470b7083dccbd3c1fc3f2
URL:
https://github.com/llvm/llvm-project/commit/fa326c21919faea33b0470b7083dccbd3c1fc3f2
DIFF:
https://github.com/llvm/llvm-project/commit/fa326c21919faea33b0470b7083dccbd3c1fc3f2.diff
L
Author: Pavel Kosov
Date: 2023-03-20T17:33:04+03:00
New Revision: 21cd04c46fe0a2bee224899f56518a09bce5306e
URL:
https://github.com/llvm/llvm-project/commit/21cd04c46fe0a2bee224899f56518a09bce5306e
DIFF:
https://github.com/llvm/llvm-project/commit/21cd04c46fe0a2bee224899f56518a09bce5306e.diff
L
Author: Pavel Kosov
Date: 2023-03-20T12:53:24+03:00
New Revision: 28997feb0c3ac243cb5cc89d7682993e23463ca7
URL:
https://github.com/llvm/llvm-project/commit/28997feb0c3ac243cb5cc89d7682993e23463ca7
DIFF:
https://github.com/llvm/llvm-project/commit/28997feb0c3ac243cb5cc89d7682993e23463ca7.diff
L
Author: Pavel Kosov
Date: 2023-03-14T16:14:47+03:00
New Revision: f81317a54586dbcef0c14cf512a0770e8ecaab3d
URL:
https://github.com/llvm/llvm-project/commit/f81317a54586dbcef0c14cf512a0770e8ecaab3d
DIFF:
https://github.com/llvm/llvm-project/commit/f81317a54586dbcef0c14cf512a0770e8ecaab3d.diff
L
Author: Pavel Kosov
Date: 2023-03-14T12:24:44+03:00
New Revision: 72474afa27570a0a1307f3260f0187b703aa6d84
URL:
https://github.com/llvm/llvm-project/commit/72474afa27570a0a1307f3260f0187b703aa6d84
DIFF:
https://github.com/llvm/llvm-project/commit/72474afa27570a0a1307f3260f0187b703aa6d84.diff
L
Author: Pavel Kosov
Date: 2021-10-21T02:39:10+03:00
New Revision: cb9a0dc293cf4ca451d625c6a54e491d8c11e591
URL:
https://github.com/llvm/llvm-project/commit/cb9a0dc293cf4ca451d625c6a54e491d8c11e591
DIFF:
https://github.com/llvm/llvm-project/commit/cb9a0dc293cf4ca451d625c6a54e491d8c11e591.diff
L
28 matches
Mail list logo
