[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, 4 insertions, 1 deletions

diff --git a/test_disassembler.cc b/test_disassembler.cc
index 8d59a93..2d6727b 100644
--- a/test_disassembler.cc
+++ b/test_disassembler.cc
@@ -8,13 +8,16 @@
 
 namespace zucchini {
 
+// |num_equivalence_iterations_| = 2 to cover common case for testing.
 TestDisassembler::TestDisassembler(const ReferenceTypeTraits& traits1,
                                    const std::vector<Reference>& refs1,
                                    const ReferenceTypeTraits& traits2,
                                    const std::vector<Reference>& refs2,
                                    const ReferenceTypeTraits& traits3,
                                    const std::vector<Reference>& refs3)
-    : traits_{traits1, traits2, traits3}, refs_{refs1, refs2, refs3} {}
+    : Disassembler(2),
+      traits_{traits1, traits2, traits3},
+      refs_{refs1, refs2, refs3} {}
 
 TestDisassembler::~TestDisassembler() = default;