From 49d685684a0a8f5a5b33fa7de3cb6c290d343cb8 Mon Sep 17 00:00:00 2001 From: "sgjesse@chromium.org" Date: Thu, 22 Apr 2010 07:35:43 +0000 Subject: [PATCH] Minor formatting changes. Review URL: http://codereview.chromium.org/1766005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@4470 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/arm/macro-assembler-arm.cc | 9 ++++----- src/ia32/codegen-ia32.cc | 6 ++---- src/x64/codegen-x64.cc | 3 +-- 3 files changed, 7 insertions(+), 11 deletions(-) diff --git a/src/arm/macro-assembler-arm.cc b/src/arm/macro-assembler-arm.cc index f8896b8..17d6a10 100644 --- a/src/arm/macro-assembler-arm.cc +++ b/src/arm/macro-assembler-arm.cc @@ -117,11 +117,10 @@ void MacroAssembler::Call(intptr_t target, RelocInfo::Mode rmode, // ldr ip, [pc, #...] // blx ip - { // NOLINT - // The two instructions (ldr and blx) could be separated by a constant - // pool and the code would still work. The issue comes from the - // patching code which expect the ldr to be just above the blx. - BlockConstPoolScope block_const_pool(this); + // The two instructions (ldr and blx) could be separated by a constant + // pool and the code would still work. The issue comes from the + // patching code which expect the ldr to be just above the blx. + { BlockConstPoolScope block_const_pool(this); // Statement positions are expected to be recorded when the target // address is loaded. The mov method will automatically record // positions when pc is the target, since this is not the case here diff --git a/src/ia32/codegen-ia32.cc b/src/ia32/codegen-ia32.cc index f8c040a..0ffe29a 100644 --- a/src/ia32/codegen-ia32.cc +++ b/src/ia32/codegen-ia32.cc @@ -154,8 +154,7 @@ void CodeGenerator::Generate(CompilationInfo* info) { #endif // New scope to get automatic timing calculation. - { // NOLINT - HistogramTimerScope codegen_timer(&Counters::code_generation); + { HistogramTimerScope codegen_timer(&Counters::code_generation); CodeGenState state(this); // Entry: @@ -9810,8 +9809,7 @@ void TranscendentalCacheStub::Generate(MacroAssembler* masm) { __ j(zero, &runtime_call_clear_stack); #ifdef DEBUG // Check that the layout of cache elements match expectations. - { // NOLINT - doesn't like a single brace on a line. - TranscendentalCache::Element test_elem[2]; + { TranscendentalCache::Element test_elem[2]; char* elem_start = reinterpret_cast(&test_elem[0]); char* elem2_start = reinterpret_cast(&test_elem[1]); char* elem_in0 = reinterpret_cast(&(test_elem[0].in[0])); diff --git a/src/x64/codegen-x64.cc b/src/x64/codegen-x64.cc index a8c823e..d14b390 100644 --- a/src/x64/codegen-x64.cc +++ b/src/x64/codegen-x64.cc @@ -304,8 +304,7 @@ void CodeGenerator::Generate(CompilationInfo* info) { #endif // New scope to get automatic timing calculation. - { // NOLINT - HistogramTimerScope codegen_timer(&Counters::code_generation); + { HistogramTimerScope codegen_timer(&Counters::code_generation); CodeGenState state(this); // Entry: -- 2.7.4