Revision: 21735
Author: [email protected]
Date: Tue Jun 10 09:24:00 2014 UTC
Log: Make presubmit script happy again.
No clue why this seems to be an issue only for me, though...
[email protected]
Review URL: https://codereview.chromium.org/321123002
http://code.google.com/p/v8/source/detail?r=21735
Modified:
/branches/bleeding_edge/include/v8-util.h
/branches/bleeding_edge/samples/process.cc
/branches/bleeding_edge/src/base/build_config.h
/branches/bleeding_edge/src/jsregexp.cc
/branches/bleeding_edge/src/mips/assembler-mips.cc
/branches/bleeding_edge/src/mips/assembler-mips.h
/branches/bleeding_edge/src/mips/macro-assembler-mips.cc
=======================================
--- /branches/bleeding_edge/include/v8-util.h Tue May 27 09:31:06 2014 UTC
+++ /branches/bleeding_edge/include/v8-util.h Tue Jun 10 09:24:00 2014 UTC
@@ -154,7 +154,7 @@
*/
bool SetReturnValue(const K& key,
ReturnValue<Value> returnValue) {
- return SetReturnValueFromVal(returnValue, Traits::Get(&impl_, key));
+ return SetReturnValueFromVal(&returnValue, Traits::Get(&impl_, key));
}
/**
@@ -227,7 +227,7 @@
}
template<typename T>
bool SetReturnValue(ReturnValue<T> returnValue) {
- return SetReturnValueFromVal(returnValue, value_);
+ return SetReturnValueFromVal(&returnValue, value_);
}
void Reset() {
value_ = kPersistentContainerNotFound;
@@ -309,10 +309,10 @@
}
static bool SetReturnValueFromVal(
- ReturnValue<Value>& returnValue, PersistentContainerValue value) {
+ ReturnValue<Value>* returnValue, PersistentContainerValue value) {
bool hasValue = value != kPersistentContainerNotFound;
if (hasValue) {
- returnValue.SetInternal(
+ returnValue->SetInternal(
*reinterpret_cast<internal::Object**>(FromVal(value)));
}
return hasValue;
=======================================
--- /branches/bleeding_edge/samples/process.cc Thu May 8 06:52:35 2014 UTC
+++ /branches/bleeding_edge/samples/process.cc Tue Jun 10 09:24:00 2014 UTC
@@ -574,7 +574,7 @@
void ParseOptions(int argc,
char* argv[],
- map<string, string>& options,
+ map<string, string>* options,
string* file) {
for (int i = 1; i < argc; i++) {
string arg = argv[i];
@@ -584,7 +584,7 @@
} else {
string key = arg.substr(0, index);
string value = arg.substr(index+1);
- options[key] = value;
+ (*options)[key] = value;
}
}
}
@@ -646,7 +646,7 @@
v8::V8::InitializeICU();
map<string, string> options;
string file;
- ParseOptions(argc, argv, options, &file);
+ ParseOptions(argc, argv, &options, &file);
if (file.empty()) {
fprintf(stderr, "No script was specified.\n");
return 1;
=======================================
--- /branches/bleeding_edge/src/base/build_config.h Tue Jun 3 08:12:43
2014 UTC
+++ /branches/bleeding_edge/src/base/build_config.h Tue Jun 10 09:24:00
2014 UTC
@@ -58,7 +58,7 @@
// Target architecture detection. This may be set externally. If not,
detect
// in the same way as the host architecture, that is, target the native
// environment as presented by the compiler.
-#if !V8_TARGET_ARCH_X64 && !V8_TARGET_ARCH_IA32 && !V8_TARGET_ARCH_X87 &&\
+#if !V8_TARGET_ARCH_X64 && !V8_TARGET_ARCH_IA32 && !V8_TARGET_ARCH_X87 && \
!V8_TARGET_ARCH_ARM && !V8_TARGET_ARCH_ARM64 && !V8_TARGET_ARCH_MIPS
#if defined(_M_X64) || defined(__x86_64__)
#define V8_TARGET_ARCH_X64 1
=======================================
--- /branches/bleeding_edge/src/jsregexp.cc Tue Jun 3 15:45:38 2014 UTC
+++ /branches/bleeding_edge/src/jsregexp.cc Tue Jun 10 09:24:00 2014 UTC
@@ -1208,8 +1208,9 @@
const OutSet& registers_to_pop,
const OutSet& registers_to_clear) {
for (int reg = max_register; reg >= 0; reg--) {
- if (registers_to_pop.Get(reg)) assembler->PopRegister(reg);
- else if (registers_to_clear.Get(reg)) {
+ if (registers_to_pop.Get(reg)) {
+ assembler->PopRegister(reg);
+ } else if (registers_to_clear.Get(reg)) {
int clear_to = reg;
while (reg > 0 && registers_to_clear.Get(reg - 1)) {
reg--;
=======================================
--- /branches/bleeding_edge/src/mips/assembler-mips.cc Tue Jun 3 15:45:38
2014 UTC
+++ /branches/bleeding_edge/src/mips/assembler-mips.cc Tue Jun 10 09:24:00
2014 UTC
@@ -1185,7 +1185,7 @@
}
-//-------Data-processing-instructions---------
+// -------Data-processing-instructions---------
// Arithmetic.
@@ -1328,7 +1328,7 @@
}
-//------------Memory-instructions-------------
+// ------------Memory-instructions-------------
// Helper for base-reg + offset, when offset is larger than int16.
void Assembler::LoadRegPlusOffsetToAt(const MemOperand& src) {
@@ -1445,7 +1445,7 @@
}
-//-------------Misc-instructions--------------
+// -------------Misc-instructions--------------
// Break / Trap instructions.
void Assembler::break_(uint32_t code, bool break_as_stop) {
@@ -1618,7 +1618,7 @@
}
-//--------Coprocessor-instructions----------------
+// --------Coprocessor-instructions----------------
// Load, store, move.
void Assembler::lwc1(FPURegister fd, const MemOperand& src) {
=======================================
--- /branches/bleeding_edge/src/mips/assembler-mips.h Tue Jun 3 08:12:43
2014 UTC
+++ /branches/bleeding_edge/src/mips/assembler-mips.h Tue Jun 10 09:24:00
2014 UTC
@@ -646,7 +646,7 @@
void jal_or_jalr(int32_t target, Register rs);
- //-------Data-processing-instructions---------
+ // -------Data-processing-instructions---------
// Arithmetic.
void addu(Register rd, Register rs, Register rt);
@@ -684,7 +684,7 @@
void rotrv(Register rd, Register rt, Register rs);
- //------------Memory-instructions-------------
+ // ------------Memory-instructions-------------
void lb(Register rd, const MemOperand& rs);
void lbu(Register rd, const MemOperand& rs);
@@ -700,12 +700,12 @@
void swr(Register rd, const MemOperand& rs);
- //----------------Prefetch--------------------
+ // ----------------Prefetch--------------------
void pref(int32_t hint, const MemOperand& rs);
- //-------------Misc-instructions--------------
+ // -------------Misc-instructions--------------
// Break / Trap instructions.
void break_(uint32_t code, bool break_as_stop = false);
@@ -738,7 +738,7 @@
void ins_(Register rt, Register rs, uint16_t pos, uint16_t size);
void ext_(Register rt, Register rs, uint16_t pos, uint16_t size);
- //--------Coprocessor-instructions----------------
+ // --------Coprocessor-instructions----------------
// Load, store, and move.
void lwc1(FPURegister fd, const MemOperand& src);
=======================================
--- /branches/bleeding_edge/src/mips/macro-assembler-mips.cc Tue Jun 3
19:23:10 2014 UTC
+++ /branches/bleeding_edge/src/mips/macro-assembler-mips.cc Tue Jun 10
09:24:00 2014 UTC
@@ -867,7 +867,7 @@
}
-//------------Pseudo-instructions-------------
+// ------------Pseudo-instructions-------------
void MacroAssembler::Ulw(Register rd, const MemOperand& rs) {
lwr(rd, rs);
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
