Re: r279765 - Omit column info for CodeView by default

2016-08-29 Thread Adrian McCarthy via cfe-commits
Sorry, no, I don't have plans to implement column ranges at this time. On Mon, Aug 29, 2016 at 2:10 PM, Adrian Prantl wrote: > > > On Aug 25, 2016, at 11:24 AM, Adrian McCarthy via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > > > > Author: amccarth > > Date: Thu Aug 25 13:24:35 2016 > >

Re: r279765 - Omit column info for CodeView by default

2016-08-29 Thread Adrian Prantl via cfe-commits
> On Aug 25, 2016, at 11:24 AM, Adrian McCarthy via cfe-commits > wrote: > > Author: amccarth > Date: Thu Aug 25 13:24:35 2016 > New Revision: 279765 > > URL: http://llvm.org/viewvc/llvm-project?rev=279765&view=rev > Log: > Omit column info for CodeView by default > > Clang tracks only start

Re: r279765 - Omit column info for CodeView by default

2016-08-29 Thread David Blaikie via cfe-commits
ah, fair enough - only saw the diff, not the rest of the file On Mon, Aug 29, 2016 at 9:59 AM Adrian McCarthy wrote: > Thanks. I was trying to be consistent with how the prefixes were used in > this particular file, where the prefix was modeled after the case-sensitive > flags being tested. > >

Re: r279765 - Omit column info for CodeView by default

2016-08-29 Thread Adrian McCarthy via cfe-commits
Thanks. I was trying to be consistent with how the prefixes were used in this particular file, where the prefix was modeled after the case-sensitive flags being tested. On Mon, Aug 29, 2016 at 8:49 AM, David Blaikie wrote: > > > On Thu, Aug 25, 2016 at 11:32 AM Adrian McCarthy via cfe-commits <

Re: r279765 - Omit column info for CodeView by default

2016-08-29 Thread David Blaikie via cfe-commits
On Thu, Aug 25, 2016 at 11:32 AM Adrian McCarthy via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: amccarth > Date: Thu Aug 25 13:24:35 2016 > New Revision: 279765 > > URL: http://llvm.org/viewvc/llvm-project?rev=279765&view=rev > Log: > Omit column info for CodeView by default > > Cl