rnk added a comment.

I think @compnerd arranged things this way. I don't know why 
-flto-visibility-public-std is involved, I have no idea what that does. I think 
we should do what MSVC does here, which is to leave _CxxThrowException 
unannotated and let the linker synthesize import thunks.

Here is the code in question that applies dllimport: 
https://github.com/llvm-git-prototype/llvm/blob/master/clang/lib/CodeGen/CodeGenModule.cpp#L2955

My suggestion would be to make it check `isWindowsItaniumEnvironment` so we can 
avoid this behavior which isn't desired under either mingw or MSVC.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55229



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

Reply via email to