Re: r316278 - [libclang, bindings]: add spelling location

2017-10-23 Thread Masud Rahman via cfe-commits
I created a new revision to review and to fix the tests: https://reviews.llvm.org/D39217. On Sat, Oct 21, 2017 at 6:24 PM, Masud Rahman wrote: > Thanks, I will take a look. > > On Sat, Oct 21, 2017 at 5:53 PM, Aaron Ballman > wrote: > >> I've reverted back to green in r316279 due to more bots f

Re: r316278 - [libclang, bindings]: add spelling location

2017-10-21 Thread Masud Rahman via cfe-commits
Thanks, I will take a look. On Sat, Oct 21, 2017 at 5:53 PM, Aaron Ballman wrote: > I've reverted back to green in r316279 due to more bots failing. > > ~Aaron > > On Sat, Oct 21, 2017 at 5:35 PM, Aaron Ballman > wrote: > > This commit appears to have broken several bots. Can you revert or > >

Re: r316278 - [libclang, bindings]: add spelling location

2017-10-21 Thread Aaron Ballman via cfe-commits
I've reverted back to green in r316279 due to more bots failing. ~Aaron On Sat, Oct 21, 2017 at 5:35 PM, Aaron Ballman wrote: > This commit appears to have broken several bots. Can you revert or > quickly fix the issue? > > http://lab.llvm.org:8011/builders/clang-ppc64be-linux/builds/11896 > htt

Re: r316278 - [libclang, bindings]: add spelling location

2017-10-21 Thread Aaron Ballman via cfe-commits
This commit appears to have broken several bots. Can you revert or quickly fix the issue? http://lab.llvm.org:8011/builders/clang-ppc64be-linux/builds/11896 http://lab.llvm.org:8011/builders/clang-s390x-linux/builds/12380 Thanks! ~Aaron On Sat, Oct 21, 2017 at 4:53 PM, Masud Rahman via cfe-comm

r316278 - [libclang, bindings]: add spelling location

2017-10-21 Thread Masud Rahman via cfe-commits
Author: frutiger Date: Sat Oct 21 13:53:49 2017 New Revision: 316278 URL: http://llvm.org/viewvc/llvm-project?rev=316278&view=rev Log: [libclang, bindings]: add spelling location o) Add a 'Location' class that represents the four properties of a physical location o) Enhance 'SourceLocation