[Zucchini] Zucchini-gen: Make number of CreateEquivalenceMap() generations depend on Disassembler.

The number of CreateEquivalenceMap() iterations used be constant
kNumIteraitons = 2. This CL makes the value depend on architecture.
Current assignment:
- DisassemblerNoOp: 1, since no pointers are identified (though in this
  case, CreateEquivalenceMap() should not be called).
- DisassemblerWin32: 2.
Upcoming DisassemblerDex will use 4.

Also applying generic cleanups on headers and comments.

Bug: 729154
Change-Id: Ia12d98fcba500e4c81c8a5d356ce4cadf424ffde
Reviewed-on: https://chromium-review.googlesource.com/961273
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Samuel Huang <huangs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542919}
NOKEYCHECK=True
GitOrigin-RevId: 55aea0a875b80e614464fdd157d9717471f9d64f

1 files changed, 2 insertions, 2 deletions

diff --git a/image_index.h b/image_index.h
index 4f07015..6301b3a 100644
--- a/image_index.h
+++ b/image_index.h
@@ -43,14 +43,14 @@ class ImageIndex {
   size_t TypeCount() const {
     if (reference_sets_.empty())
       return 0U;
-    return reference_sets_.rbegin()->first.value() + 1;
+    return reference_sets_.rbegin()->first.value() + 1;  // Max key + 1.
   }
 
   // Returns the array size needed to accommodate all pool values.
   size_t PoolCount() const {
     if (target_pools_.empty())
       return 0U;
-    return target_pools_.rbegin()->first.value() + 1;
+    return target_pools_.rbegin()->first.value() + 1;  // Max key + 1.
   }
 
   // Returns true if |image_[location]| is either: