[PATCH] D27592: Reimplement depedency tracking in the ImplicitNullChecks pass

[Anna Thomas via Phabricator via llvm-commits]

anna added inline comments.


================
Comment at: lib/CodeGen/ImplicitNullChecks.cpp:374
 
-  HazardDetector HD(*TRI, *AA);
+  SmallVector<MachineInstr *, 8> InstsSeenSoFar;
 
----------------
sanjoy wrote:
> anna wrote:
> > Just curious if there's any reason for choosing 8? I think the usual number for any sort of llvm related threshold is 6?
> No reason -- 8 just seemed "more than enough".  I can change it to 6 if feel strongly about it.
8 is fine.


https://reviews.llvm.org/D27592