[cfe-commits] [Windows] Properly mangle templates
    Timur Iskhodzhanov 
    timurrrr at google.com
       
    Thu May 24 04:35:04 PDT 2012
    
    
  
On Thu, May 24, 2012 at 2:44 PM, João Matos <ripzonetriton at gmail.com> wrote:
>>> Attached is a patch that introduces template mangling (at least in the
>>> simple cases) as well as some tests for the code.
>>>
>>> Can you please review it?
> I'm only wondering if there is any other template argument kind we need to handle.
We'll find out as we compile more code with Clang.
Currently, even simple programs with templates are miscompiled, the
suggested patch should cover most of the cases.
I hope all other cases will hit the assertion and then we can deal
with minimized test cases.
The current patch (plus one more patch to handle back referenes)
mangles templates from googletest and iostream just fine (at least
without obvious problems).
> The patch looks fine to me.
Can I commit or should I wait for John's review?
> --
> João Matos
    
    
More information about the cfe-commits
mailing list