This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG203cd01b54dd: [PowerPC] [NFC] Add Big-Endian checks for
existing MMA tests (authored by saghir).
Repository:
rG LLVM Github Monorepo
CHANGES SINC
saghir updated this revision to Diff 372143.
saghir added a comment.
Rebased before commit.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109126/new/
https://reviews.llvm.org/D109126
Files:
clang/test/CodeGen/builtins-ppc-pair-mma.c
clang/test
saghir updated this revision to Diff 371919.
saghir added a comment.
Rebased.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109126/new/
https://reviews.llvm.org/D109126
Files:
clang/test/CodeGen/builtins-ppc-pair-mma.c
clang/test/Sema/ppc-pair
saghir marked an inline comment as done.
saghir added inline comments.
Comment at: clang/test/CodeGen/builtins-ppc-pair-mma.c:5
+// RUN: %clang_cc1 -O3 -triple powerpc64-unknown-unknown -target-cpu pwr10 \
+// RUN: -emit-llvm %s -o - | FileCheck %s --check-prefix=CHECK-BE
-
saghir updated this revision to Diff 371164.
saghir added a comment.
Addressed review comments.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109126/new/
https://reviews.llvm.org/D109126
Files:
clang/test/CodeGen/builtins-ppc-pair-mma.c
clang/
lei added a comment.
Is it really necessary to add the BE checks in this patch if they are the same
as LE checks? Why not just add it later when there is a diff seen?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109126/new/
https://reviews.llvm.
nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.
LGTM. Please update the description and commit message as to why this is being
added.
Comment at: clang/test/CodeGen/builtins-ppc-pair-mma.c:5
+// RUN: %clang_cc1 -O3 -t
qiucf added inline comments.
Comment at: clang/test/CodeGen/builtins-ppc-pair-mma.c:5
+// RUN: %clang_cc1 -O3 -triple powerpc64-unknown-unknown -target-cpu pwr10 \
+// RUN: -emit-llvm %s -o - | FileCheck %s --check-prefix=CHECK-BE
Seems just adding `%clang_cc1
saghir created this revision.
Herald added subscribers: shchenz, kbarton, nemanjai.
saghir requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This patch adds Big-Endian checks for the existing MMA test cases.
Repository:
rG LLVM Github Mon
