diff options
author | Ian Rogers <irogers@google.com> | 2014-10-31 00:33:20 -0700 |
---|---|---|
committer | Ian Rogers <irogers@google.com> | 2014-11-03 20:01:04 -0800 |
commit | 6a3c1fcb4ba42ad4d5d142c17a3712a6ddd3866f (patch) | |
tree | 9df58b57af13240a93a6da4eefcf03f70cce9ad9 /runtime/gc/collector | |
parent | c6e0955737e15f7c0c3575d4e13789b3411f4993 (diff) | |
download | art-6a3c1fcb4ba42ad4d5d142c17a3712a6ddd3866f.zip art-6a3c1fcb4ba42ad4d5d142c17a3712a6ddd3866f.tar.gz art-6a3c1fcb4ba42ad4d5d142c17a3712a6ddd3866f.tar.bz2 |
Remove -Wno-unused-parameter and -Wno-sign-promo from base cflags.
Fix associated errors about unused paramenters and implict sign conversions.
For sign conversion this was largely in the area of enums, so add ostream
operators for the effected enums and fix tools/generate-operator-out.py.
Tidy arena allocation code and arena allocated data types, rather than fixing
new and delete operators.
Remove dead code.
Change-Id: I5b433e722d2f75baacfacae4d32aef4a828bfe1b
Diffstat (limited to 'runtime/gc/collector')
-rw-r--r-- | runtime/gc/collector/concurrent_copying.h | 4 | ||||
-rw-r--r-- | runtime/gc/collector/mark_sweep.cc | 1 | ||||
-rw-r--r-- | runtime/gc/collector/sticky_mark_sweep.cc | 1 |
3 files changed, 5 insertions, 1 deletions
diff --git a/runtime/gc/collector/concurrent_copying.h b/runtime/gc/collector/concurrent_copying.h index ce7c75a..ee5a785 100644 --- a/runtime/gc/collector/concurrent_copying.h +++ b/runtime/gc/collector/concurrent_copying.h @@ -29,7 +29,9 @@ class ConcurrentCopying : public GarbageCollector { const std::string& name_prefix = "") : GarbageCollector(heap, name_prefix + (name_prefix.empty() ? "" : " ") + - "concurrent copying + mark sweep") {} + "concurrent copying + mark sweep") { + UNUSED(generational); + } ~ConcurrentCopying() {} diff --git a/runtime/gc/collector/mark_sweep.cc b/runtime/gc/collector/mark_sweep.cc index ad3bb11..e3966e3 100644 --- a/runtime/gc/collector/mark_sweep.cc +++ b/runtime/gc/collector/mark_sweep.cc @@ -658,6 +658,7 @@ class MarkStackTask : public Task { // Scans all of the objects virtual void Run(Thread* self) SHARED_LOCKS_REQUIRED(Locks::mutator_lock_) EXCLUSIVE_LOCKS_REQUIRED(Locks::heap_bitmap_lock_) { + UNUSED(self); ScanObjectParallelVisitor visitor(this); // TODO: Tune this. static const size_t kFifoSize = 4; diff --git a/runtime/gc/collector/sticky_mark_sweep.cc b/runtime/gc/collector/sticky_mark_sweep.cc index 4ed6abc..5be3db7 100644 --- a/runtime/gc/collector/sticky_mark_sweep.cc +++ b/runtime/gc/collector/sticky_mark_sweep.cc @@ -58,6 +58,7 @@ void StickyMarkSweep::MarkReachableObjects() { } void StickyMarkSweep::Sweep(bool swap_bitmaps) { + UNUSED(swap_bitmaps); SweepArray(GetHeap()->GetLiveStack(), false); } |