http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc File src/arm/code-stubs-arm.cc (right):
http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode1710 src/arm/code-stubs-arm.cc:1710: Handle<Code> GetUnaryOpStub(int key, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode1810 src/arm/code-stubs-arm.cc:1810: void UnaryOpStub::GenerateSmiCodeSub(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode1825 src/arm/code-stubs-arm.cc:1825: void UnaryOpStub::GenerateSmiCodeBitNot(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode1863 src/arm/code-stubs-arm.cc:1863: void UnaryOpStub::GenerateHeapNumberStubBitNot( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits on one line?
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode1873 src/arm/code-stubs-arm.cc:1873: void UnaryOpStub::GenerateHeapNumberCodeSub(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode1991 src/arm/code-stubs-arm.cc:1991: void UnaryOpStub::GenerateGenericCodeFallback( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits on one line?
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode2008 src/arm/code-stubs-arm.cc:2008: Handle<Code> GetBinaryOpStub(int key, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Full indentation (not 4 spaces) or int key on a separate line.
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode2096 src/arm/code-stubs-arm.cc:2096: void BinaryOpStub::GenerateSmiSmiOperation( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits one line?
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode2222 src/arm/code-stubs-arm.cc:2222: void BinaryOpStub::GenerateFPOperation(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode2436 src/arm/code-stubs-arm.cc:2436: void BinaryOpStub::GenerateSmiCode(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
MacroAssembler* masm, should go on a separate line.
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode2766 src/arm/code-stubs-arm.cc:2766: (result_type_ <= BinaryOpIC::INT32) ? &transition On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode2769 src/arm/code-stubs-arm.cc:2769: __ b(mi, (result_type_ <= BinaryOpIC::INT32) ? &transition On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/arm/code-stubs-arm.cc#newcode2972 src/arm/code-stubs-arm.cc:2972: void BinaryOpStub::GenerateHeapResultAllocation( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits full indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/ia32/code-stubs-ia32.cc File src/ia32/code-stubs-ia32.cc (right): http://codereview.chromium.org/7063017/diff/1/src/ia32/code-stubs-ia32.cc#newcode520 src/ia32/code-stubs-ia32.cc:520: Handle<Code> GetUnaryOpStub(int key, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/ia32/code-stubs-ia32.cc#newcode623 src/ia32/code-stubs-ia32.cc:623: void UnaryOpStub::GenerateSmiCodeSub(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/ia32/code-stubs-ia32.cc#newcode707 src/ia32/code-stubs-ia32.cc:707: void UnaryOpStub::GenerateHeapNumberCodeSub(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/ia32/code-stubs-ia32.cc#newcode743 src/ia32/code-stubs-ia32.cc:743: void UnaryOpStub::GenerateHeapNumberCodeBitNot( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Move MacroAssembler* masm, up one line and full indent of Label* slow?
Done. http://codereview.chromium.org/7063017/diff/1/src/ia32/code-stubs-ia32.cc#newcode840 src/ia32/code-stubs-ia32.cc:840: void UnaryOpStub::GenerateGenericCodeFallback( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits one line?
Done. http://codereview.chromium.org/7063017/diff/1/src/ia32/code-stubs-ia32.cc#newcode892 src/ia32/code-stubs-ia32.cc:892: void BinaryOpStub::GenerateTypeTransitionWithSavedArgs( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits one line?
Done. http://codereview.chromium.org/7063017/diff/1/src/ia32/code-stubs-ia32.cc#newcode970 src/ia32/code-stubs-ia32.cc:970: void BinaryOpStub::GenerateSmiCode(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
MacroAssembler* masm, on a separate line
Done. http://codereview.chromium.org/7063017/diff/1/src/ic.cc File src/ic.cc (right): http://codereview.chromium.org/7063017/diff/1/src/ic.cc#newcode2203 src/ic.cc:2203: UnaryOpIC::TypeInfo UnaryOpIC::ComputeNewType( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Possibly reorganize arguments.
Done. http://codereview.chromium.org/7063017/diff/1/src/ic.cc#newcode2264 src/ic.cc:2264: BinaryOpIC::TypeInfo BinaryOpIC::JoinTypes(BinaryOpIC::TypeInfo x, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/ic.cc#newcode2279 src/ic.cc:2279: BinaryOpIC::TypeInfo BinaryOpIC::GetTypeInfo(Handle<Object> left, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/ic.cc#newcode2373 src/ic.cc:2373: Handle<Code> GetBinaryOpStub(int key, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc File src/mips/code-stubs-mips.cc (right): http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc#newcode1907 src/mips/code-stubs-mips.cc:1907: void UnaryOpStub::GenerateSmiCodeSub(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc#newcode1922 src/mips/code-stubs-mips.cc:1922: void UnaryOpStub::GenerateSmiCodeBitNot(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc#newcode1960 src/mips/code-stubs-mips.cc:1960: void UnaryOpStub::GenerateHeapNumberStubBitNot( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits one line?
Done. http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc#newcode1970 src/mips/code-stubs-mips.cc:1970: void UnaryOpStub::GenerateHeapNumberCodeSub(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc#newcode2003 src/mips/code-stubs-mips.cc:2003: void UnaryOpStub::GenerateHeapNumberCodeBitNot( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Reorganize arguments?
Done. http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc#newcode2104 src/mips/code-stubs-mips.cc:2104: Handle<Code> GetBinaryOpStub(int key, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Full indentation or int key on separate line.
Done. http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc#newcode2193 src/mips/code-stubs-mips.cc:2193: void BinaryOpStub::GenerateSmiSmiOperation( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits one line?
Done. http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc#newcode2348 src/mips/code-stubs-mips.cc:2348: void BinaryOpStub::GenerateFPOperation(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/mips/code-stubs-mips.cc#newcode2564 src/mips/code-stubs-mips.cc:2564: void BinaryOpStub::GenerateSmiCode(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
MacroAssembler* masm, on separate line.
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc File src/x64/code-stubs-x64.cc (right): http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode409 src/x64/code-stubs-x64.cc:409: Handle<Code> GetUnaryOpStub(int key, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode487 src/x64/code-stubs-x64.cc:487: void UnaryOpStub::GenerateSmiCodeSub(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode501 src/x64/code-stubs-x64.cc:501: void UnaryOpStub::GenerateSmiCodeBitNot( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits full intentation?
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode549 src/x64/code-stubs-x64.cc:549: void UnaryOpStub::GenerateHeapNumberCodeSub(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Indentation.
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode590 src/x64/code-stubs-x64.cc:590: void UnaryOpStub::GenerateHeapNumberCodeBitNot( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits full indentation?
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode643 src/x64/code-stubs-x64.cc:643: void UnaryOpStub::GenerateGenericCodeFallback( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits one line?
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode684 src/x64/code-stubs-x64.cc:684: Handle<Code> GetBinaryOpStub(int key, On 2011/05/24 11:33:11, Søren Gjesse wrote:
Full intentation or int key on separate line.
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode773 src/x64/code-stubs-x64.cc:773: void BinaryOpStub::GenerateSmiCode(MacroAssembler* masm, On 2011/05/24 11:33:11, Søren Gjesse wrote:
MacroAssembler* masm, on separate line.
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode923 src/x64/code-stubs-x64.cc:923: void BinaryOpStub::GenerateFloatingPointCode( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits full indentation?
Done. http://codereview.chromium.org/7063017/diff/1/src/x64/code-stubs-x64.cc#newcode1228 src/x64/code-stubs-x64.cc:1228: void BinaryOpStub::GenerateHeapResultAllocation( On 2011/05/24 11:33:11, Søren Gjesse wrote:
Fits full indentation?
Done. http://codereview.chromium.org/7063017/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
