This revision was automatically updated to reflect the committed changes.
Closed by commit rG62f5ba624bfb: [PowerPC][Power10] Implement Test LSB by Byte
Builtins in LLVM/Clang (authored by amyk).
Changed prior to commit:
https://reviews.llvm.org/D82431?vs=274667=277646#toc
Repository:
rG
lei accepted this revision.
lei added a comment.
LGTM thx!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82431/new/
https://reviews.llvm.org/D82431
___
cfe-commits mailing list
amyk updated this revision to Diff 274667.
amyk added a comment.
Update patch to:
- place MC tests in correct files
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82431/new/
https://reviews.llvm.org/D82431
Files:
lei added a comment.
encoding tests need to be placed in corresponding `ppc64-encoding-ISA31*` files
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82431/new/
https://reviews.llvm.org/D82431
___
steven.zhang added inline comments.
Comment at: llvm/test/MC/Disassembler/PowerPC/vsx.txt:2
# RUN: llvm-mc --disassemble %s -triple powerpc64-unknown-linux-gnu -mcpu=pwr7
| FileCheck %s
+# RUN: llvm-mc --disassemble %s -triple powerpc64-unknown-linux-gnu \
+# RUN:
amyk updated this revision to Diff 274239.
amyk added a comment.
Updated revision to:
- add the MC test cases into `vsx.s` and `vsx.txt`.
- create a test file called `builtins-ppc-p10vsx.ll` to place all Power10 VSX
builtins tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST
amyk added a comment.
@lei Could you please take another look at this to see if this change is OK?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82431/new/
https://reviews.llvm.org/D82431
___
lei accepted this revision as: lei.
lei added a comment.
This revision is now accepted and ready to land.
LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82431/new/
https://reviews.llvm.org/D82431
___
cfe-commits mailing list
amyk updated this revision to Diff 273745.
amyk added a comment.
Updated to the correct indentation for the functions in `altivec.h`.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82431/new/
https://reviews.llvm.org/D82431
Files:
clang/include/clang/Basic/BuiltinsPPC.def
amyk created this revision.
amyk added reviewers: nemanjai, lei, saghir, hfinkel, power-llvm-team, PowerPC.
amyk added projects: LLVM, clang, PowerPC.
Herald added subscribers: shchenz, hiraditya.
This patch implements builtins for the following prototypes:
int vec_test_lsbb_all_ones (vector
10 matches
Mail list logo