phoebewang wrote:
> > > Doesn't this break ABI by changing intrinsic / builtin numbers?
> >
> >
> > So the headers could still be updated but must use the existing builtins in
> > the backport?
>
> Sounds good, I can try with it. Thanks!
A reduced one for intrinsics only
https://github.com/
https://github.com/phoebewang closed
https://github.com/llvm/llvm-project/pull/133219
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
phoebewang wrote:
> > Doesn't this break ABI by changing intrinsic / builtin numbers?
>
> So the headers could still be updated but must use the existing builtins in
> the backport?
Sounds good, I can try with it. Thanks!
https://github.com/llvm/llvm-project/pull/133219
__
https://github.com/phoebewang approved this pull request.
https://github.com/llvm/llvm-project/pull/134331
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
phoebewang wrote:
> @phoebewang What do you think about merging this PR to the release branch?
I think it's ok to merge.
https://github.com/llvm/llvm-project/pull/134331
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://l
phoebewang wrote:
> Although these instructions aren't supported by current hardware this
> probably needs a release notes entry - wdyt?
Sure, added in the description. Thanks for the reminder!
https://github.com/llvm/llvm-project/pull/133219
___
llv
https://github.com/phoebewang edited
https://github.com/llvm/llvm-project/pull/133219
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
phoebewang wrote:
This is a bug fix without risk, LGTM.
https://github.com/llvm/llvm-project/pull/126687
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
phoebewang wrote:
@e-kud Please help review it.
> It had the 'needs review' status and I missed it. We can get it into -rc2.
Oh, I thought it notified reviewer automaticly. Maybe because it's manully
cherry-pick.
https://github.com/llvm/llvm-project/pull/125057
___
phoebewang wrote:
Ping @tstellar, any reason this was missing in rc1 release?
https://github.com/llvm/llvm-project/pull/125057
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-
https://github.com/phoebewang updated
https://github.com/llvm/llvm-project/pull/125057
>From f816bd39f6986825e338198fce8747939ab1c882 Mon Sep 17 00:00:00 2001
From: Phoebe Wang
Date: Thu, 30 Jan 2025 21:13:49 +0800
Subject: [PATCH] [X86][AVX10] Disable m[no-]avx10.1 and switch m[no-]avx10.2
to
https://github.com/phoebewang updated
https://github.com/llvm/llvm-project/pull/125057
>From f816bd39f6986825e338198fce8747939ab1c882 Mon Sep 17 00:00:00 2001
From: Phoebe Wang
Date: Thu, 30 Jan 2025 21:13:49 +0800
Subject: [PATCH] [X86][AVX10] Disable m[no-]avx10.1 and switch m[no-]avx10.2
to
https://github.com/phoebewang milestoned
https://github.com/llvm/llvm-project/pull/125057
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/phoebewang created
https://github.com/llvm/llvm-project/pull/125057
Per the feedback we got, we’d like to switch m[no-]avx10.2 to alias of 512 bit
options and disable m[no-]avx10.1 due to they were alias of 256 bit options.
We also change -mno-avx10.[1,2]-512 to alias of 256
phoebewang wrote:
LGTM.
https://github.com/llvm/llvm-project/pull/111246
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
phoebewang wrote:
Thanks @tru! The title should be good as the description.
https://github.com/llvm/llvm-project/pull/109767
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-br
https://github.com/phoebewang edited
https://github.com/llvm/llvm-project/pull/109767
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/phoebewang closed
https://github.com/llvm/llvm-project/pull/109635
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
phoebewang wrote:
Sorry, there was a mistake with the patch. I'll close it and create another one.
https://github.com/llvm/llvm-project/pull/109635
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
https://github.com/phoebewang milestoned
https://github.com/llvm/llvm-project/pull/109635
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/phoebewang created
https://github.com/llvm/llvm-project/pull/109635
Promoted KMOV* was encoded with CD8 incorrectly, see
https://godbolt.org/z/cax513hG1
>From b403d2a05b548f24b46bab4c4ae014c9949f3c44 Mon Sep 17 00:00:00 2001
From: Phoebe Wang
Date: Mon, 23 Sep 2024 09:41:43
@@ -0,0 +1,17 @@
+; RUN: llc -verify-machineinstrs < %s -relocation-model=pic
+
+target datalayout =
"e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
phoebewang wrote:
Yes, we need to check the
@@ -0,0 +1,17 @@
+; RUN: llc -verify-machineinstrs < %s -relocation-model=pic
+
+target datalayout =
"e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
phoebewang wrote:
Do not need this.
https://github.com/llvm/llvm-project/pull/106965
__
@@ -0,0 +1,17 @@
+; RUN: llc -verify-machineinstrs < %s -relocation-model=pic
+
+target datalayout =
"e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
+
+; Passing a pointer to thread-local storage to a function c
@@ -0,0 +1,17 @@
+; RUN: llc -verify-machineinstrs < %s -relocation-model=pic
+
+target datalayout =
"e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
phoebewang wrote:
This can put in the RUN li
https://github.com/phoebewang approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/105815
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/phoebewang approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/105638
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch
phoebewang wrote:
This patch fixes build failures when compiling AVX512 code using
`-march=native` on machines without AVX512. The problem was introduced by
https://github.com/llvm/llvm-project/commit/a7b8b890600a33e0c88d639f311f1d73ccb1c8d2
which is included in LLVM 18.1.5 release.
https://g
phoebewang wrote:
> @phoebewang (or anyone else). If you would like to add a note about this fix
> in the release notes (completely optional). Please reply to this comment with
> a one or two sentence description of the fix. When you are done, please add
> the release:note label to this PR.
T
phoebewang wrote:
> @phoebewang (or anyone else). If you would like to add a note about this fix
> in the release notes (completely optional). Please reply to this comment with
> a one or two sentence description of the fix. When you are done, please add
> the release:note label to this PR.
T
phoebewang wrote:
The test failures are caused by LLVM 18 branch difference, created #91425
instead.
https://github.com/llvm/llvm-project/pull/91161
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin
https://github.com/phoebewang closed
https://github.com/llvm/llvm-project/pull/91161
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/phoebewang milestoned
https://github.com/llvm/llvm-project/pull/91425
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/phoebewang created
https://github.com/llvm/llvm-project/pull/91425
None
>From 5f3651376c051c1fb7b29741778a4616811a1157 Mon Sep 17 00:00:00 2001
From: Phoebe Wang
Date: Mon, 6 May 2024 10:59:44 +0800
Subject: [PATCH 1/2] [X86][FP16] Do not create VBROADCAST_LOAD for f16 witho
phoebewang wrote:
> Hi @phoebewang (or anyone else). If you would like to add a note about this
> fix in the release notes (completely optional). Please reply to this comment
> with a one or two sentence description of the fix. When you are done, please
> add the release:note label to this PR.
phoebewang wrote:
> @phoebewang What do you think about backporting this?
I think the patch doesn't fix all problem in #86371, suggest to reevaluate it.
@efriedma-quic may take another look.
https://github.com/llvm/llvm-project/pull/86698
___
llvm-br
phoebewang wrote:
> @phoebewang What do you think about backporting this?
I didn't review on it. Maybe @topperc can evaluate it.
https://github.com/llvm/llvm-project/pull/86728
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
ht
https://github.com/phoebewang approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/88388
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/phoebewang updated
https://github.com/llvm/llvm-project/pull/83834
>From 9cec3b7b1ea0491df688555a51750efe6c9bd075 Mon Sep 17 00:00:00 2001
From: Phoebe Wang
Date: Mon, 4 Mar 2024 18:09:41 +0800
Subject: [PATCH] [X86] Add missing subvector_subreg_lowering for BF16 (#83720)
--
https://github.com/phoebewang updated
https://github.com/llvm/llvm-project/pull/83834
>From 6d03789303fe3d5c84406df29353ddf63199eb08 Mon Sep 17 00:00:00 2001
From: Phoebe Wang
Date: Mon, 4 Mar 2024 18:09:41 +0800
Subject: [PATCH] [X86] Add missing subvector_subreg_lowering for BF16 (#83720)
--
phoebewang wrote:
Please review this patch instead https://github.com/llvm/llvm-project/pull/83834
https://github.com/llvm/llvm-project/pull/84491
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
https://github.com/phoebewang updated
https://github.com/llvm/llvm-project/pull/83834
>From 6d03789303fe3d5c84406df29353ddf63199eb08 Mon Sep 17 00:00:00 2001
From: Phoebe Wang
Date: Mon, 4 Mar 2024 18:09:41 +0800
Subject: [PATCH] [X86] Add missing subvector_subreg_lowering for BF16 (#83720)
--
phoebewang wrote:
LGTM.
https://github.com/llvm/llvm-project/pull/84698
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
phoebewang wrote:
I don't think it worth backport. The x87 usage is not very common nowadays, and
the problem is lasting for a while.
I'd suggest not do the backport considering it may have sideeffect in the
isCall change.
https://github.com/llvm/llvm-project/pull/84058
___
phoebewang wrote:
Risk is minimum, LGTM.
https://github.com/llvm/llvm-project/pull/84029
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/phoebewang milestoned
https://github.com/llvm/llvm-project/pull/83834
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/phoebewang created
https://github.com/llvm/llvm-project/pull/83834
None
>From 6d03789303fe3d5c84406df29353ddf63199eb08 Mon Sep 17 00:00:00 2001
From: Phoebe Wang
Date: Mon, 4 Mar 2024 18:09:41 +0800
Subject: [PATCH] [X86] Add missing subvector_subreg_lowering for BF16 (#8372
https://github.com/phoebewang created
https://github.com/llvm/llvm-project/pull/83811
None
>From 7210a98061d9f6382e8ce4ca7646ec4a5c3f96e6 Mon Sep 17 00:00:00 2001
From: Phoebe Wang
Date: Mon, 4 Mar 2024 17:28:23 +0800
Subject: [PATCH] [X86] Add missing subvector_subreg_lowering for BF16 (#8372
phoebewang wrote:
Some change related on trunk code, I'll create a manual cherry-pick.
https://github.com/llvm/llvm-project/pull/83758
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
https://github.com/phoebewang closed
https://github.com/llvm/llvm-project/pull/83758
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
phoebewang wrote:
> @phoebewang What do you think about merging this PR to the release branch?
This patch fixes prior mistakes, so should be merged to release branch.
@KanRobert Do I understand it right?
https://github.com/llvm/llvm-project/pull/79721
__
phoebewang wrote:
> @phoebewang What do you think about merging this PR to the release branch?
The fix is required and has low risk, LGTM.
https://github.com/llvm/llvm-project/pull/79341
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.l
Author: Phoebe Wang
Date: 2023-11-19T10:14:34+08:00
New Revision: 5237193b87721134541f228e28edfd544a9c8ac8
URL:
https://github.com/llvm/llvm-project/commit/5237193b87721134541f228e28edfd544a9c8ac8
DIFF:
https://github.com/llvm/llvm-project/commit/5237193b87721134541f228e28edfd544a9c8ac8.diff
L
Author: Phoebe Wang
Date: 2022-08-04T22:31:16+08:00
New Revision: e4b3fad1facdb3f005ec77f607a2b05e3e9fcbad
URL:
https://github.com/llvm/llvm-project/commit/e4b3fad1facdb3f005ec77f607a2b05e3e9fcbad
DIFF:
https://github.com/llvm/llvm-project/commit/e4b3fad1facdb3f005ec77f607a2b05e3e9fcbad.diff
L
Author: Phoebe Wang
Date: 2021-11-13T11:30:09+08:00
New Revision: e7b1dc9a9dbabf0a13a76d44b179c0230222
URL:
https://github.com/llvm/llvm-project/commit/e7b1dc9a9dbabf0a13a76d44b179c0230222
DIFF:
https://github.com/llvm/llvm-project/commit/e7b1dc9a9dbabf0a13a76d44b179c0230222.diff
L
Author: Phoebe Wang
Date: 2021-11-13T11:32:57+08:00
New Revision: a566a4b21157e7aedb6d4230f61e09f6cd3152ce
URL:
https://github.com/llvm/llvm-project/commit/a566a4b21157e7aedb6d4230f61e09f6cd3152ce
DIFF:
https://github.com/llvm/llvm-project/commit/a566a4b21157e7aedb6d4230f61e09f6cd3152ce.diff
L
56 matches
Mail list logo