[PATCH] D26064: [ADT] IntervalMap: fix setStart and setStop
Vedant Kumar via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 28 10:59:09 PDT 2016
vsk added a comment.
In https://reviews.llvm.org/D26064#582164, @mlemay-intel wrote:
> In https://reviews.llvm.org/D26064#581794, @vsk wrote:
>
> > ...
> >
> > - If we create a 0-element interval using setStart/setStop, it just gets deleted.
>
>
> The revised assertion will fail in that case. Is deleting the interval actually the desired behavior?
Looking at it more, it shouldn't be possible to do what I described (creating a 0-element interval using setStart/setStop). I'm walking the request for that test back :).
https://reviews.llvm.org/D26064
More information about the llvm-commits
mailing list