[PATCH] Allow dllimport/dllexport on inline functions and get the linkage right

Hans Wennborg hans at chromium.org
Thu May 15 11:51:36 PDT 2014


Regarding handling the dll linkage in AST or Codegen, I was kind of hoping doing it in AST would help in functions such as ASTContext::DeclMustBeEmitted (but it currently doesn't).

I don't have enough experience here to feel strongly either way, but I do suspect that it will become clearer which solution is best as we get further along the train of patches towards full support for class-level dllimport/dllexport.

I would like to go ahead and commit this now, and am totally open to moving the linkage adjusting code if it seems like a good idea as we go forward.

http://reviews.llvm.org/D3772






More information about the cfe-commits mailing list