You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2023/01/16 06:17:25 UTC
[kudu] 02/02: [codegen] update CodegenTest.TestDumpMC for ppc64le
This is an automated email from the ASF dual-hosted git repository.
alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit 04fac32072a5bc760485e7a3048f8e8e6ae44239
Author: harinreddy <hn...@us.ibm.com>
AuthorDate: Sun Sep 18 22:31:12 2022 -0500
[codegen] update CodegenTest.TestDumpMC for ppc64le
The equivalent of x86_64's "retq" assembler instruction on ppc64le
is "blr".
Change-Id: I81933185665a131200ea8ac4e0cc323e6226a355
Reviewed-on: http://gerrit.cloudera.org:8080/19015
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <al...@apache.org>
---
src/kudu/codegen/codegen-test.cc | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/kudu/codegen/codegen-test.cc b/src/kudu/codegen/codegen-test.cc
index 8d5bc9d31..dfde2297f 100644
--- a/src/kudu/codegen/codegen-test.cc
+++ b/src/kudu/codegen/codegen-test.cc
@@ -382,11 +382,13 @@ TEST_F(CodegenTest, TestDumpMC) {
const vector<string>& msgs = sink.logged_msgs();
ASSERT_EQ(msgs.size(), 1);
- #ifndef __aarch64__
- EXPECT_THAT(msgs[0], testing::ContainsRegex("retq"));
- #else
+#if defined(__powerpc__)
+ EXPECT_THAT(msgs[0], testing::ContainsRegex("blr"));
+#elif defined(__aarch64__)
EXPECT_THAT(msgs[0], testing::ContainsRegex("ret"));
- #endif //__aarch64__
+#else
+ EXPECT_THAT(msgs[0], testing::ContainsRegex("retq"));
+#endif // #if defined(__powerpc__) ... #elif defined(__aarch64__) ... #else ...
}
// Basic test for the CompilationManager code cache.