[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