Hi John, Can you please review this patch?
It fixes the mangling ( http://llvm.org/bugs/show_bug.cgi?id=13207 ), adds some comments as well as adding a lot of tests for the mangler. I've intentionally added a few extra namespaces to the test to improve readability or to cover more codepaths. I hope the new comments in the code are good enough so I don't need to write the details in this e-mail. FYI, I've kept some mangler tests until PR13389 is reviewed/fixed. I'll add more tests as a separate patch when both patches are reviewed and committed. By the way, all of the not-yet-shared tests pass when both patches are applied. Thanks, Timur Iskhodzhanov, Google Russia
bug_13207_args.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
