[PATCH] D85802: [clang] Add -fc++-abi= flag for specifying which C++ ABI to use

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 17 17:27:39 PDT 2022


rjmccall added a comment.

It's certainly possible that other targets will want to support multiple C++ ABIs in the future, but it's okay for us to design things around the situation we've got today.  A lot of these "ABIs" are just target-specific variants; if it simplifies code to just make ABI queries just be target queries instead of abstracting the target C++ ABI, and the resulting burden on targets with multiple supported ABIs is very small, that seems like an acceptable trade-off.  If things change and we get a lot of targets asking to support multiple ABIs, there's no reason we can't revisit this decision.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85802



More information about the cfe-commits mailing list