>>! In D4663#4, @rengolin wrote:
> Hi Eli,
> 
> Looks ok to me, but I'm not the expert. Would be good to have some negative 
> tests, though.
> 
> cheers,
> --renato

I'm not 100% what you mean as "negative tests" in this context. I did add a 
check that nodes that shouldn't get mangled, aren't, in this new patch. Also 
addressed the other review by checking MS mangling as well.

http://reviews.llvm.org/D4663



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to