[PATCH] Unified type trait parsing

Alp Toker alp at nuanti.com
Thu Dec 12 13:27:55 PST 2013


On 12/12/2013 20:13, Richard Smith wrote:
> Looks great, thanks!

r197179!

Alp.


>
>
> On Wed, Dec 11, 2013 at 10:39 PM, Alp Toker <alp at nuanti.com 
> <mailto:alp at nuanti.com>> wrote:
>
>     Hi,
>
>     Type trait parsing is all over the place at the moment with unary,
>     binary and n-ary C++11 type traits having been developed
>     independently through different points in clang's history.
>
>     There's no good reason to handle them separately -- there are
>     three parsers, three AST nodes and lots of duplicated handling
>     code with slightly different implementations and diags for each kind.
>
>     The attached patch unifies parsing of type traits and sets the
>     stage for further consolidation.
>
>     No change in behaviour other than slightly more consistent error
>     recovery.
>
>      9 files changed, 141 insertions(+), 330 deletions(-)
>
>     Alp.
>
>
>     -- 
>     http://www.nuanti.com
>     the browser experts
>
>
>     _______________________________________________
>     cfe-commits mailing list
>     cfe-commits at cs.uiuc.edu <mailto:cfe-commits at cs.uiuc.edu>
>     http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>

-- 
http://www.nuanti.com
the browser experts




More information about the cfe-commits mailing list