mcbarton wrote:

> I think that it might be better to wait on this change until we do this 
> properly. In the mean time, the workaround would be to use 
> `-fvisibility=default`. If you want to help with the annotations to make 
> clang build as a DSO, I think that @andrurogerz would welcome the help. But, 
> this change should be split up so that it can be easily reviewed similar to 
> how we have been approaching the LLVM side.

I'll avoid using fvisibility=default and stick witht he current export method, 
as using   fvisibility=default globally increased the shared library size from 
an already big 45Mb to an even bigger for 66 Mb. Having the symbols annotated 
individual allowed CppInterOp to keep the same size. Hopefully with 
@andrurogerz work to annotate these interfaces, CppInterOp shared library will 
automatically get the added benefit of these all these symbols becoming visible 
😊

https://github.com/llvm/llvm-project/pull/146786
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to