Re: r264417 - [ASTMatchers] Add own version of VariadicFunction.

2016-04-26 Thread Alexander Kornienko via cfe-commits
BTW, this commit removes the last usage of llvm/ADT/VariadicFunction.h. Should we kill this header? On Fri, Mar 25, 2016 at 8:51 PM, Mehdi Amini via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Thanks! > > On Mar 25, 2016, at 12:47 PM, Samuel Benzaquen wrote: > > Fixed in r264453. > Sorry

Re: r264417 - [ASTMatchers] Add own version of VariadicFunction.

2016-03-25 Thread Mehdi Amini via cfe-commits
Thanks! > On Mar 25, 2016, at 12:47 PM, Samuel Benzaquen wrote: > > Fixed in r264453. > Sorry for the inconvenience. > > _Sam > > On Fri, Mar 25, 2016 at 3:31 PM, Samuel Benzaquen > wrote: > Sorry, this is an unrelated problem. > It is using brace-init lists and the

Re: r264417 - [ASTMatchers] Add own version of VariadicFunction.

2016-03-25 Thread Samuel Benzaquen via cfe-commits
Fixed in r264453. Sorry for the inconvenience. _Sam On Fri, Mar 25, 2016 at 3:31 PM, Samuel Benzaquen wrote: > Sorry, this is an unrelated problem. > It is using brace-init lists and the supported MSVC version does not > support them. > I'll fix that. > > > On Fri, Mar 25, 2016 at 3:27 PM, Mehd

Re: r264417 - [ASTMatchers] Add own version of VariadicFunction.

2016-03-25 Thread Samuel Benzaquen via cfe-commits
Sorry, this is an unrelated problem. It is using brace-init lists and the supported MSVC version does not support them. I'll fix that. On Fri, Mar 25, 2016 at 3:27 PM, Mehdi Amini wrote: > The link I provided is testing r264430... > > -- > Mehdi > > On Mar 25, 2016, at 12:25 PM, Samuel Benzaquen

Re: r264417 - [ASTMatchers] Add own version of VariadicFunction.

2016-03-25 Thread Mehdi Amini via cfe-commits
The link I provided is testing r264430... -- Mehdi > On Mar 25, 2016, at 12:25 PM, Samuel Benzaquen wrote: > > I believe r264428 fixes this problem. > > On Fri, Mar 25, 2016 at 2:18 PM, Mehdi Amini > wrote: > Hi, > > I think this broke clang-tidy somehow: > ht

Re: r264417 - [ASTMatchers] Add own version of VariadicFunction.

2016-03-25 Thread Samuel Benzaquen via cfe-commits
I believe r264428 fixes this problem. On Fri, Mar 25, 2016 at 2:18 PM, Mehdi Amini wrote: > Hi, > > I think this broke clang-tidy somehow: > http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/10881/steps/build%20stage%201/logs/stdio > > -- > Mehdi > > > > > > On Mar 25, 2016, at 9:29

Re: r264417 - [ASTMatchers] Add own version of VariadicFunction.

2016-03-25 Thread Mehdi Amini via cfe-commits
Hi, I think this broke clang-tidy somehow: http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/10881/steps/build%20stage%201/logs/stdio -- Mehdi > On Mar 25, 2016, at 9:29 AM, Samuel Benzaquen via cfe-commits > wrote: > > Author: sbenza > Date: Fri Mar 25 11:29:30 2016 > New Re