lebedev.ri added a comment.

In D59214#1429422 <https://reviews.llvm.org/D59214#1429422>, @riccibruno wrote:

> In D59214#1429400 <https://reviews.llvm.org/D59214#1429400>, @lebedev.ri 
> wrote:
>
> > In D59214#1429384 <https://reviews.llvm.org/D59214#1429384>, @riccibruno 
> > wrote:
> >
> > > IIRC, last time I looked only 4 statement/expression classes currently 
> > > have some abbreviation defined.
> >
> >
> > Yep, that is what i'm seeing in this diff.
> >
> > In D59214#1429384 <https://reviews.llvm.org/D59214#1429384>, @riccibruno 
> > wrote:
> >
> > > It would probably be useful to have someone go systematically through the 
> > > list of classes and fix this.
> >
> >
> > Yeah, i suspect that might be **really** beneficial.
>
>
> Especially given that more people are going to use modules now that they are 
> in the process of being standardized.


Ah, good point, filed https://bugs.llvm.org/show_bug.cgi?id=41072


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59214/new/

https://reviews.llvm.org/D59214



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

Reply via email to