[PATCH] D18756: [Coverage] Avoid unnecessary copying of an array. NFC.

[Vedant Kumar via llvm-commits]

vsk added a subscriber: vsk.
vsk added a comment.

I agree with dblaikie, the API could be a bit cleaner. Wdyt of this:

1. Have the SegmentBuilder constructor accept a `vector<CoverageSegment>&`; assert that it's empty.
2. Replace `Segments = SegmentBuilder().buildSegments(...)` with `SegmentBuilder(Segments).build(...)`.


http://reviews.llvm.org/D18756