Prazek added a comment.

In http://reviews.llvm.org/D19201#445496, @aaron.ballman wrote:

> In http://reviews.llvm.org/D19201#445406, @sbarzowski wrote:
>
> > Note ``FunctionProtoType::getNoExceptExpr`` is weird. If you have the same 
> > expr in multiple noexcepts, then it returns the same object for all of 
> > them, so it is useless for determining the location in code.
> >
> > I ended up finding it all "by hand" with lexer (looking for matching 
> > parentheses).
>
>
> I think you may want to take a dependency on: http://reviews.llvm.org/D20428


We will see how much time it will take to make it pushable to master. IMHO this 
code is not complicated enough to make it wait for your patch, but it's cool 
that you took care of it :)


Repository:
  rL LLVM

http://reviews.llvm.org/D19201



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to