[PATCH] [ms-cxxabi] Mangle variadic template parameter packs

[Reid Kleckner]

  Thanks, committing since this is pretty simple.


================
Comment at: lib/AST/MicrosoftMangle.cpp:137
@@ -136,3 +136,3 @@
                           const TemplateArgumentList &TemplateArgs);
-
+  void mangleTemplateArg(const TemplateDecl *TD, const TemplateArgument &TA, int i);
 };
----------------
Timur Iskhodzhanov wrote:
> Can you come up with a more descriptive name?
> e.g. OrdinalNumber?
Sure, ArgIndex.

================
Comment at: lib/AST/MicrosoftMangle.cpp:878
@@ +877,3 @@
+  case TemplateArgument::Pack:
+    // MSVC 2013 is the first version to support variadic templates, and it
+    // doesn't do anything special to mangle template parameter packs.
----------------
Timur Iskhodzhanov wrote:
> I think it's safe to remove this comment
I want to keep it because it's a divergence from Itanium, which has a special character code.


http://llvm-reviews.chandlerc.com/D1077

COMMIT
  http://llvm-reviews.chandlerc.com/rL185454