Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(840)

Unified Diff: src/mips64/simulator-mips64.h

Issue 1133163005: MIPS64: Enable shorten-64-to-32 warning. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Fix type related to mach_timespec. Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/mips64/macro-assembler-mips64.cc ('k') | src/mips64/simulator-mips64.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/mips64/simulator-mips64.h
diff --git a/src/mips64/simulator-mips64.h b/src/mips64/simulator-mips64.h
index aa76c94ad9e1b5626d6f2cc804767cc435c2462f..346d3584f44f4c3eb718623cf45b5c8d87d6fa47 100644
--- a/src/mips64/simulator-mips64.h
+++ b/src/mips64/simulator-mips64.h
@@ -326,57 +326,55 @@ class Simulator {
inline int32_t SetDoubleLOW(double* addr);
// functions called from DecodeTypeRegister
- void DecodeTypeRegisterCOP1(Instruction* instr, const int32_t& rs_reg,
- const int64_t& rs, const uint64_t& rs_u,
- const int32_t& rt_reg, const int64_t& rt,
- const uint64_t& rt_u, const int32_t& rd_reg,
- const int32_t& fr_reg, const int32_t& fs_reg,
- const int32_t& ft_reg, const int32_t& fd_reg,
+ void DecodeTypeRegisterCOP1(Instruction* instr, const int32_t rs_reg,
+ const int64_t rs, const uint64_t rs_u,
+ const int32_t rt_reg, const int64_t rt,
+ const uint64_t rt_u, const int32_t rd_reg,
+ const int32_t fr_reg, const int32_t fs_reg,
+ const int32_t ft_reg, const int32_t fd_reg,
int64_t& alu_out);
- void DecodeTypeRegisterCOP1X(Instruction* instr, const int32_t& fr_reg,
- const int32_t& fs_reg, const int32_t& ft_reg,
- const int32_t& fd_reg);
+ void DecodeTypeRegisterCOP1X(Instruction* instr, const int32_t fr_reg,
+ const int32_t fs_reg, const int32_t ft_reg,
+ const int32_t fd_reg);
void DecodeTypeRegisterSPECIAL(
- Instruction* instr, const int64_t& rs_reg, const int64_t& rs,
- const uint64_t& rs_u, const int64_t& rt_reg, const int64_t& rt,
- const uint64_t& rt_u, const int64_t& rd_reg, const int32_t& fr_reg,
- const int32_t& fs_reg, const int32_t& ft_reg, const int64_t& fd_reg,
- int64_t& i64hilo, uint64_t& u64hilo, int64_t& alu_out, bool& do_interrupt,
- int64_t& current_pc, int64_t& next_pc, int64_t& return_addr_reg,
- int64_t& i128resultH, int64_t& i128resultL);
+ Instruction* instr, const int32_t rs_reg, const int64_t rs,
+ const uint64_t rs_u, const int32_t rt_reg, const int64_t rt,
+ const uint64_t rt_u, const int32_t rd_reg, const int32_t fr_reg,
+ const int32_t fs_reg, const int32_t ft_reg, const int32_t fd_reg,
+ const int64_t i64hilo, const uint64_t u64hilo, const int64_t alu_out,
+ const bool do_interrupt, const int64_t current_pc, const int64_t next_pc,
+ const int32_t return_addr_reg, const int64_t i128resultH,
+ const int64_t i128resultL);
- void DecodeTypeRegisterSPECIAL2(Instruction* instr, const int64_t& rd_reg,
- int64_t& alu_out);
- void DecodeTypeRegisterSPECIAL3(Instruction* instr, const int64_t& rt_reg,
- const int64_t& rd_reg, int64_t& alu_out);
+ void DecodeTypeRegisterSPECIAL2(Instruction* instr, const int32_t rd_reg,
+ const int64_t alu_out);
- void DecodeTypeRegisterSRsType(Instruction* instr, const int32_t& fs_reg,
- const int32_t& ft_reg, const int32_t& fd_reg);
+ void DecodeTypeRegisterSPECIAL3(Instruction* instr, const int32_t rt_reg,
+ const int32_t rd_reg, const int64_t alu_out);
- void DecodeTypeRegisterDRsType(Instruction* instr, const int32_t& fs_reg,
- const int32_t& ft_reg, const int32_t& fd_reg);
+ void DecodeTypeRegisterSRsType(Instruction* instr, const int32_t fs_reg,
+ const int32_t ft_reg, const int32_t fd_reg);
- void DecodeTypeRegisterWRsType(Instruction* instr, const int32_t& fs_reg,
- const int32_t& ft_reg, const int32_t& fd_reg,
+ void DecodeTypeRegisterDRsType(Instruction* instr, const int32_t fs_reg,
+ const int32_t ft_reg, const int32_t fd_reg);
+
+ void DecodeTypeRegisterWRsType(Instruction* instr, const int32_t fs_reg,
+ const int32_t ft_reg, const int32_t fd_reg,
int64_t& alu_out);
- void DecodeTypeRegisterLRsType(Instruction* instr, const int32_t& fs_reg,
- const int32_t& fd_reg, const int32_t& ft_reg);
+ void DecodeTypeRegisterLRsType(Instruction* instr, const int32_t fs_reg,
+ const int32_t fd_reg, const int32_t ft_reg);
// Executing is handled based on the instruction type.
void DecodeTypeRegister(Instruction* instr);
// Helper function for DecodeTypeRegister.
- void ConfigureTypeRegister(Instruction* instr,
- int64_t* alu_out,
- int64_t* i64hilo,
- uint64_t* u64hilo,
- int64_t* next_pc,
- int64_t* return_addr_reg,
- bool* do_interrupt,
- int64_t* result128H,
+ void ConfigureTypeRegister(Instruction* instr, int64_t* alu_out,
+ int64_t* i64hilo, uint64_t* u64hilo,
+ int64_t* next_pc, int* return_addr_reg,
+ bool* do_interrupt, int64_t* result128H,
int64_t* result128L);
void DecodeTypeImmediate(Instruction* instr);
@@ -418,7 +416,7 @@ class Simulator {
// ICache.
static void CheckICache(v8::internal::HashMap* i_cache, Instruction* instr);
static void FlushOnePage(v8::internal::HashMap* i_cache, intptr_t start,
- int size);
+ size_t size);
static CachePage* GetCachePage(v8::internal::HashMap* i_cache, void* page);
enum Exception {
@@ -498,12 +496,14 @@ class Simulator {
#ifdef MIPS_ABI_N64
#define CALL_GENERATED_REGEXP_CODE(entry, p0, p1, p2, p3, p4, p5, p6, p7, p8) \
- Simulator::current(Isolate::Current())->Call( \
- entry, 10, p0, p1, p2, p3, p4, p5, p6, p7, NULL, p8)
+ static_cast<int>( \
+ Simulator::current(Isolate::Current()) \
+ ->Call(entry, 10, p0, p1, p2, p3, p4, p5, p6, p7, NULL, p8))
#else // Must be O32 Abi.
#define CALL_GENERATED_REGEXP_CODE(entry, p0, p1, p2, p3, p4, p5, p6, p7, p8) \
- Simulator::current(Isolate::Current())->Call( \
- entry, 10, p0, p1, p2, p3, NULL, p4, p5, p6, p7, p8)
+ static_cast<int>( \
+ Simulator::current(Isolate::Current()) \
+ ->Call(entry, 10, p0, p1, p2, p3, NULL, p4, p5, p6, p7, p8))
#endif // MIPS_ABI_N64
« no previous file with comments | « src/mips64/macro-assembler-mips64.cc ('k') | src/mips64/simulator-mips64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698