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

[David Blaikie via Phabricator via cfe-commits]

dblaikie added a comment.
Herald added a subscriber: abrachet.
Herald added a project: All.

FWIW, the existence of this flag and the TargetCXXABI abstraction (which predates the flag, but might otherwise be more simply factored/implemented with inheritance rather than switch tables) came up in discussion D135326 <https://reviews.llvm.org/D135326> and D119051 <https://reviews.llvm.org/D119051>.

(eg: the RecordLayoutBuilders are split by Itanium and Microsoft - but there's loads of triple checks throughout including for C++ layout choices (eg: `getBaseOrPreferredBaseAlignFromUnpacked`, empty base handling, etc) that aren't captured by the TargetCXXABI abstraction)

& so the question was where to put this next C++ Itanium ABI fix which is currently applied to Itanium in general, except for PS, Darwin, and Clang ABI <= 15.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85802