[PATCH] D93922: Mangle `__alignof__` differently than `alignof`.

Louis Dionne via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 12 10:40:08 PST 2021


ldionne added inline comments.


================
Comment at: clang/lib/AST/ItaniumMangle.cpp:4009
+    // mangling. Previously, it used a special-cased nonstandard extension.
+    if (Context.getASTContext().getLangOpts().getClangABICompat() >=
+        LangOptions::ClangABI::Ver11) {
----------------
Please forgive my ignorance, but when left unspecified, what does the Clang ABI compatibility flag default to? I'm sure you've thought about it already, but I'm trying to understand whether that is an ABI break for people compiling without specifying the Clang ABI compatibility version (which is most users).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93922



More information about the cfe-commits mailing list