commit a5359f7caabf418dc8b4978e789d908174324c46
author Vladimir Marko <vmarko@google.com> Mon Feb 18 16:29:43 2019 +0000
committer Vladimir Marko <vmarko@google.com> Mon Feb 18 16:31:06 2019 +0000
tree 4b2557ac6aa243f47e7a031bb5709fcfbe0052d7
parent ae1d2c8a1b6cc36a079c429661b3c6195de48c92

ART: Delete obsolete comments in GVN.

GVN is now using ScopedArenaAllocator and explicitly
filling allocated memory with zeros as needed.

Test: Rely on TreeHugger.
Change-Id: Ic2059014ee4ab9157a8cce64ba0206f9b276481e

compiler/optimizing/gvn.cc

diff --git a/compiler/optimizing/gvn.cc b/compiler/optimizing/gvn.cc
index 3689d1d..e8460a8 100644
--- a/compiler/optimizing/gvn.cc
+++ b/compiler/optimizing/gvn.cc
@@ -43,7 +43,6 @@
         buckets_(allocator->AllocArray<Node*>(num_buckets_, kArenaAllocGvn)),
         buckets_owned_(allocator, num_buckets_, false, kArenaAllocGvn),
         num_entries_(0u) {
-    // ArenaAllocator returns zeroed memory, so no need to set buckets to null.
     DCHECK(IsPowerOfTwo(num_buckets_));
     std::fill_n(buckets_, num_buckets_, nullptr);
     buckets_owned_.SetInitialBits(num_buckets_);
@@ -57,8 +56,6 @@
         buckets_(allocator->AllocArray<Node*>(num_buckets_, kArenaAllocGvn)),
         buckets_owned_(allocator, num_buckets_, false, kArenaAllocGvn),
         num_entries_(0u) {
-    // ArenaAllocator returns zeroed memory, so entries of buckets_ and
-    // buckets_owned_ are initialized to null and false, respectively.
     DCHECK(IsPowerOfTwo(num_buckets_));
     PopulateFromInternal(other);
   }