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.