MIPS: Fixing CLANG compilation warnings

test/cctest/test-assembler-mips64.cc

Index: test/cctest/test-assembler-mips64.cc
diff --git a/test/cctest/test-assembler-mips64.cc b/test/cctest/test-assembler-mips64.cc
index 67143e61d40529135e0fe3007522a94a8c756112..8188154e8e4ee26a1a5cca9a105301f888e9d03e 100644
--- a/test/cctest/test-assembler-mips64.cc
+++ b/test/cctest/test-assembler-mips64.cc
@@ -4629,7 +4629,8 @@ uint64_t run_align(uint64_t rs_value, uint64_t rt_value, uint8_t bp) {
   F2 f = FUNCTION_CAST<F2>(code->entry());
 
   uint64_t res = reinterpret_cast<uint64_t>(
-      CALL_GENERATED_CODE(isolate, f, rs_value, rt_value, 0, 0, 0));
+      CALL_GENERATED_CODE(isolate, f, static_cast<int>(rs_value),
+        static_cast<int>(rt_value), 0, 0, 0));

[balazs.kilvady, 2015/12/02 18:49:07]

If F2 function type (with int params) is used above then why we need uint64_t
rs_value, rt_value function args insted of int in run_align(uint64_t rs_value,
uint64_t rt_value, uint8_t bp)? If run_align would get int params then we don't
need reinterpret_cast<>s. Would not that work? Or if 64 bit alignment must be
tested then F4 function type should be used, I guess.

[ivica.bogosavljevic, 2015/12/03 10:36:24]

Acknowledged.

 
   return res;
 }
@@ -5720,7 +5721,7 @@ uint64_t run_dsll(uint64_t rt_value, uint16_t sa_value) {
   F2 f = FUNCTION_CAST<F2>(code->entry());
 
   uint64_t res = reinterpret_cast<uint64_t>(
-      CALL_GENERATED_CODE(isolate, f, rt_value, 0, 0, 0, 0));
+      CALL_GENERATED_CODE(isolate, f, static_cast<int>(rt_value), 0, 0, 0, 0));
 
   return res;
 }