[PATCH] D118352: [clang][ABI] New c++20 modules mangling scheme

Nathan Sidwell via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 4 03:46:54 PST 2022


urnathan added a comment.

In D118352#3358864 <https://reviews.llvm.org/D118352#3358864>, @ChuanqiXu wrote:

> If I don't misread the codes, it looks like `mangleModuleInitializer` is not called.



> Now we could add test for partitions.

Correct, it is not called as the global initializer pieces are not yet implemented.  Let's not hold up this patch for that nor remove piece that will become necessary at that point.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118352/new/

https://reviews.llvm.org/D118352



More information about the cfe-commits mailing list