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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to