[llvm] [llvm] DLLExport public methods from SmallVector (PR #127850)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 13 07:27:55 PDT 2025


nikic wrote:

> > Before you do any further work on this project, please create a new discourse thread, which explains what the further plan for adding and maintaining these annotations is. There is a bunch of scattered discussion in various random pull requests, like #109702 or #113097. As this will be a sweeping, highly visible change across the entire codebase, it is important to have a central, up-to-date description of what is going on that further PRs can reference, and make sure that people are on board with your methodology before it starts getting implemented.
> 
> FWIW, this has already had an RFC on Discourse (https://discourse.llvm.org/t/supporting-llvm-build-llvm-dylib-on-windows/58891) as well as has a meta issue for tracking progress (#109483). I agree that it's a sweeping change, but I believe the buy-in already exists and this work has been ongoing for quite a while (at least in Clang, I'm less up-to-date on the LLVM side). This is a very important project for getting plugins to work on Windows, which is a significant use case for Clang (especially our tooling, like clang-tidy), so hopefully LLVM is willing to take on this functionality as well.

To avoid sounding like a broken record, I'll just point you to this to explain what my expectation is here: https://github.com/llvm/llvm-project/pull/113097#discussion_r1853651217

Basically, all I want is that when a maintainer receives a PR adding ABI annotations (or just sees them in code), there is some coherent, up-to-date documentation on what they are supposed to do (which is largely just "nothing"), so we don't have to repeat this discussion for the (N+1)th time.

https://github.com/llvm/llvm-project/pull/127850


More information about the llvm-commits mailing list