On Jul 23, 2012, at 7:53 AM, Timur Iskhodzhanov wrote: > 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.
What a bizarre rule. This looks fine, but you might be happier using llvm::StringMap. John. _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
