[PATCH] D26064: [ADT] IntervalMap: fix setStart and setStop

Vedant Kumar via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 28 17:47:29 PDT 2016


vsk accepted this revision.
vsk added a reviewer: vsk.
vsk added a comment.
This revision is now accepted and ready to land.

LGTM. Could you wait a day or two before committing? I haven't worked on IntervalMap before, so it'd be nice to others a chance to chime in.



================
Comment at: unittests/ADT/IntervalMapTest.cpp:19
+typedef IntervalMap<unsigned, unsigned, 4,
+                    IntervalMapHalfOpenInfo<unsigned> > UUHalfOpenMap;
 
----------------
nit: You don't need the extra space, 'IntervalMapHalfOpenInfo<unsigned>>' would be fine.


https://reviews.llvm.org/D26064





More information about the llvm-commits mailing list