[PATCH] D109192: [WIP/DNM] Support: introduce public API annotation support

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 3 08:18:55 PDT 2021


compnerd added a comment.

One question that remains is the spelling of the macro.  Do we want to go with `API` or `ABI`?  I think that there is an argument to be made that `ABI` is more precise - this marks how you call into the library not just how you use it.  When a function is moved beyond the module boundary, while the API will not change, the ABI of the module would.  This of course pedantic, but these items are important to nail down correctly early as these are fairly large scale invasive changes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109192



More information about the llvm-commits mailing list