[PATCH] D7775: [IR] Avoid the need to prefix MS C++ symbols with '\01'

Rafael Avila de Espindola via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 15 18:58:30 PDT 2018


espindola accepted this revision.
espindola added a comment.
This revision is now accepted and ready to land.

LGTM with just a nit.



================
Comment at: llvm/lib/IR/Mangler.cpp:146
+    MSFunc = nullptr;
+    Prefix = '\0';
+  }
----------------
You don't need to set Prefix in here, just MSFunc. With that you also don't need the {}.


https://reviews.llvm.org/D7775





More information about the llvm-commits mailing list