diff --git a/BUILD.gn b/BUILD.gn index 57faf910721e16773ebd685442ba8d51d0222b77..f1d5724e77dbc03ab273bee6d26725a85a785205 100644 --- a/BUILD.gn +++ b/BUILD.gn @@ -154,6 +154,7 @@ source_set("libark_js_intl_static") { config("ark_jsruntime_common_config") { defines = [ "PANDA_ENABLE_LTO" ] cflags_cc = [ + "-Wall", "-pedantic", "-Wno-invalid-offsetof", "-Wno-gnu-statement-expression", @@ -273,8 +274,6 @@ config("ark_jsruntime_config") { config("ecma_test_config") { visibility = [ ":*" ] - cflags_cc = [ "-Wno-sign-compare" ] - ldflags = [ "-Wl,-rpath=\$ORIGIN/" ] } diff --git a/ecmascript/mem/concurrent_marker.cpp b/ecmascript/mem/concurrent_marker.cpp index 7f40010a79741b9194ee9a112d64045108fa807e..1eba018e419ace488f8b9d6fa32c6c74b0f81ac9 100644 --- a/ecmascript/mem/concurrent_marker.cpp +++ b/ecmascript/mem/concurrent_marker.cpp @@ -122,7 +122,7 @@ void ConcurrentMarker::InitializeMarking() heapObjectSize_ = heap_->GetHeapObjectSize(); heap_->GetOldSpace()->SelectCSet(); // The alive object size of Region in OldSpace will be recompute - heap_->EnumerateNonNewSpaceRegions([this](Region *current) { + heap_->EnumerateNonNewSpaceRegions([](Region *current) { current->ResetAliveObject(); }); } else { diff --git a/ecmascript/mem/mix_gc.cpp b/ecmascript/mem/mix_gc.cpp index 14e64288d01cecb7871449ab78da6709e5c362de..a4037e06eb7d5e9dfa22aeedf51a3c42601b65ea 100644 --- a/ecmascript/mem/mix_gc.cpp +++ b/ecmascript/mem/mix_gc.cpp @@ -57,7 +57,7 @@ void MixGC::InitializePhase() heap_->Prepare(); if (heap_->IsFullMark()) { heap_->GetOldSpace()->SelectCSet(); - heap_->EnumerateNonNewSpaceRegions([this](Region *current) { + heap_->EnumerateNonNewSpaceRegions([](Region *current) { current->ResetAliveObject(); }); } diff --git a/ecmascript/mem/parallel_evacuation-inl.h b/ecmascript/mem/parallel_evacuation-inl.h index 16ac8a3afd9efd48424953c421d27b1dc0496b84..dc771582ce146c860a1ccf67bf807b650f9b3c2c 100644 --- a/ecmascript/mem/parallel_evacuation-inl.h +++ b/ecmascript/mem/parallel_evacuation-inl.h @@ -84,7 +84,7 @@ bool ParallelEvacuation::UpdateWeakObjectSlot(TaggedObject *value, ObjectSlot &s void ParallelEvacuation::SetObjectFieldRSet(TaggedObject *object, JSHClass *cls) { Region *region = Region::ObjectAddressToRange(object); - auto callbackWithCSet = [this, region](TaggedObject *root, ObjectSlot start, ObjectSlot end) { + auto callbackWithCSet = [region](TaggedObject *root, ObjectSlot start, ObjectSlot end) { for (ObjectSlot slot = start; slot < end; slot++) { JSTaggedType value = slot.GetTaggedType(); if (JSTaggedValue(value).IsHeapObject()) { diff --git a/ecmascript/mem/parallel_evacuation.cpp b/ecmascript/mem/parallel_evacuation.cpp index 12dc8c5f67bfb8523637cc419060b35339aaadab..68f45735fe8efa92d504b243de0be84369ec2c70 100644 --- a/ecmascript/mem/parallel_evacuation.cpp +++ b/ecmascript/mem/parallel_evacuation.cpp @@ -347,7 +347,7 @@ void ParallelEvacuation::UpdateAndSweepNewRegionReference(Region *region) uintptr_t freeStart = region->GetBegin(); uintptr_t freeEnd = freeStart + region->GetAllocatedBytes(); if (markBitmap != nullptr) { - markBitmap->IterateOverMarkedChunks([this, ®ion, &freeStart](void *mem) { + markBitmap->IterateOverMarkedChunks([this, &freeStart](void *mem) { ASSERT(region->InRange(ToUintPtr(mem))); auto header = reinterpret_cast(mem); JSHClass *klass = header->GetClass(); diff --git a/ecmascript/tooling/test/BUILD.gn b/ecmascript/tooling/test/BUILD.gn index faae467f05eb9ad7272553467e060dd9db19ad8a..871470c8becccdd57051a7deb74f8683146d889d 100644 --- a/ecmascript/tooling/test/BUILD.gn +++ b/ecmascript/tooling/test/BUILD.gn @@ -22,7 +22,6 @@ module_output_path = "ark/js_runtime" config("debug_api_test") { visibility = [ ":*" ] - cflags_cc = [ "-Wno-gnu-zero-variadic-macro-arguments" ] ldflags = [ "-Wl,-rpath=\$ORIGIN/" ] include_dirs = [ "//ark/js_runtime",