Re: [Lldb-commits] [PATCH] D50290: Fix a bug in VMRange

2018-08-03 Thread Zachary Turner via lldb-commits
Looks like i was wrong, nevermind! On Fri, Aug 3, 2018 at 6:23 PM Leonard Mosescu via Phabricator via lldb-commits wrote: > lemo added subscribers: bgianfo, labath, penryu, teemperor, zturner. > lemo added a comment. > > Thanks Zach. I can't find llvm::contains() though, any pointers to it? > > >

Re: [Lldb-commits] [PATCH] D50290: Fix a bug in VMRange

2018-08-03 Thread Leonard Mosescu via lldb-commits
Thanks Zach. I can't find llvm::contains() though, any pointers to it? On Fri, Aug 3, 2018 at 5:58 PM, Zachary Turner via Phabricator via lldb-commits wrote: > zturner added a subscriber: lemo. > zturner added a comment. > > I think we have llvm::contains() which would allow you to make this > s

Re: [Lldb-commits] [PATCH] D50290: Fix a bug in VMRange

2018-08-03 Thread Zachary Turner via lldb-commits
I think we have llvm::contains() which would allow you to make this slightly better. Other than that, good find! On Fri, Aug 3, 2018 at 5:49 PM Leonard Mosescu via Phabricator < revi...@reviews.llvm.org> wrote: > lemo created this revision. > lemo added reviewers: zturner, labath, teemperor. > l