Commit e53986d2 authored by plind44@gmail.com's avatar plind44@gmail.com

MIPS: Make the mips port build cleanly with clang.

Port r13583 (b39b116b) and also fix a couple mips-specific warnings.

BUG=

Review URL: https://codereview.chromium.org/13321002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14100 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 797dd305
...@@ -291,7 +291,6 @@ class LInstruction: public ZoneObject { ...@@ -291,7 +291,6 @@ class LInstruction: public ZoneObject {
SetOncePointer<LPointerMap> pointer_map_; SetOncePointer<LPointerMap> pointer_map_;
HValue* hydrogen_value_; HValue* hydrogen_value_;
bool is_call_; bool is_call_;
bool is_save_doubles_;
}; };
......
...@@ -1928,7 +1928,7 @@ void MacroAssembler::BranchShort(Label* L, Condition cond, Register rs, ...@@ -1928,7 +1928,7 @@ void MacroAssembler::BranchShort(Label* L, Condition cond, Register rs,
BranchDelaySlot bdslot) { BranchDelaySlot bdslot) {
BRANCH_ARGS_CHECK(cond, rs, rt); BRANCH_ARGS_CHECK(cond, rs, rt);
int32_t offset; int32_t offset = 0;
Register r2 = no_reg; Register r2 = no_reg;
Register scratch = at; Register scratch = at;
if (rt.is_reg()) { if (rt.is_reg()) {
...@@ -2361,7 +2361,7 @@ void MacroAssembler::BranchAndLinkShort(Label* L, Condition cond, Register rs, ...@@ -2361,7 +2361,7 @@ void MacroAssembler::BranchAndLinkShort(Label* L, Condition cond, Register rs,
BranchDelaySlot bdslot) { BranchDelaySlot bdslot) {
BRANCH_ARGS_CHECK(cond, rs, rt); BRANCH_ARGS_CHECK(cond, rs, rt);
int32_t offset; int32_t offset = 0;
Register r2 = no_reg; Register r2 = no_reg;
Register scratch = at; Register scratch = at;
if (rt.is_reg()) { if (rt.is_reg()) {
...@@ -5514,7 +5514,6 @@ bool AreAliased(Register r1, Register r2, Register r3, Register r4) { ...@@ -5514,7 +5514,6 @@ bool AreAliased(Register r1, Register r2, Register r3, Register r4) {
CodePatcher::CodePatcher(byte* address, int instructions) CodePatcher::CodePatcher(byte* address, int instructions)
: address_(address), : address_(address),
instructions_(instructions),
size_(instructions * Assembler::kInstrSize), size_(instructions * Assembler::kInstrSize),
masm_(NULL, address, size_ + Assembler::kGap) { masm_(NULL, address, size_ + Assembler::kGap) {
// Create a new macro assembler pointing to the address of the code to patch. // Create a new macro assembler pointing to the address of the code to patch.
......
...@@ -1565,7 +1565,6 @@ class CodePatcher { ...@@ -1565,7 +1565,6 @@ class CodePatcher {
private: private:
byte* address_; // The address of the code being patched. byte* address_; // The address of the code being patched.
int instructions_; // Number of instructions of the expected patch size.
int size_; // Number of bytes of the expected patch size. int size_; // Number of bytes of the expected patch size.
MacroAssembler masm_; // Macro assembler used to generate the code. MacroAssembler masm_; // Macro assembler used to generate the code.
}; };
......
...@@ -513,7 +513,7 @@ void MipsDebugger::Debug() { ...@@ -513,7 +513,7 @@ void MipsDebugger::Debug() {
int32_t words; int32_t words;
if (argc == next_arg) { if (argc == next_arg) {
words = 10; words = 10;
} else if (argc == next_arg + 1) { } else {
if (!GetValue(argv[next_arg], &words)) { if (!GetValue(argv[next_arg], &words)) {
words = 10; words = 10;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment