This revision was automatically updated to reflect the committed changes.
Closed by commit rG113f077f808f: [X86] Pass to transform tdpbf16ps intrinsics
to scalar operation. (authored by yubing).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new
yubing updated this revision to Diff 332190.
yubing added a comment.
Rebase after https://reviews.llvm.org/D98773 is merged.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new/
https://reviews.llvm.org/D96110
Files:
llvm/lib/Target/X86/X86L
pengfei accepted this revision.
pengfei added a comment.
This revision is now accepted and ready to land.
LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new/
https://reviews.llvm.org/D96110
__
yubing updated this revision to Diff 331766.
yubing added a comment.
address Pengfei's comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new/
https://reviews.llvm.org/D96110
Files:
llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp
llvm/
yubing added inline comments.
Comment at: llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp:318-319
+// calculate idxa, idxb, idxc
+// %eltc = extractelement <256 x i32> %vec.c.inner.phi, i16 %idxc
+// %eltcf32 = bitcast i32 %eltc to float
+// %elta = extractelement <
pengfei added inline comments.
Comment at: llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp:318-319
+// calculate idxa, idxb, idxc
+// %eltc = extractelement <256 x i32> %vec.c.inner.phi, i16 %idxc
+// %eltcf32 = bitcast i32 %eltc to float
+// %elta = extractelement
yubing updated this revision to Diff 330898.
yubing added a comment.
just do a rebase
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new/
https://reviews.llvm.org/D96110
Files:
llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp
llvm/test/CodeG
yubing updated this revision to Diff 325688.
yubing added a comment.
Modify some comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new/
https://reviews.llvm.org/D96110
Files:
clang/include/clang/Basic/BuiltinsX86_64.def
clang/lib/He
yubing updated this revision to Diff 325685.
yubing added a comment.
Fix incorrect naming for dpbf16's bb
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new/
https://reviews.llvm.org/D96110
Files:
clang/include/clang/Basic/BuiltinsX86_64.de
yubing updated this revision to Diff 325683.
yubing added a comment.
Rebase and add a testcase.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new/
https://reviews.llvm.org/D96110
Files:
clang/include/clang/Basic/BuiltinsX86_64.def
clang/
yubing updated this revision to Diff 325170.
yubing edited the summary of this revision.
yubing added a comment.
Address comments above and refactor some code
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new/
https://reviews.llvm.org/D96110
pengfei added inline comments.
Comment at: llvm/test/CodeGen/X86/AMX/amx-low-intrinsics.ll:174-175
+; CHECK-NEXT:[[TMP13:%.*]] = bitcast i32 [[TMP12]] to <2 x i16>
+; CHECK-NEXT:[[TMP14:%.*]] = zext <2 x i16> [[TMP11]] to <2 x i32>
+; CHECK-NEXT:[[TMP15:%.*]] = shl <2
yubing updated this revision to Diff 321675.
yubing added a comment.
Rebase and add a testcase for dpbf16ps intrinsic.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96110/new/
https://reviews.llvm.org/D96110
Files:
clang/include/clang/Basic/Buil
yubing created this revision.
Herald added subscribers: pengfei, hiraditya.
yubing requested review of this revision.
Herald added projects: clang, LLVM.
Herald added subscribers: llvm-commits, cfe-commits.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D96110
Files:
clang/incl
14 matches
Mail list logo