[PATCH] D36501: Add flag to request Clang is ABI-compatible with older versions of itself

Paul Robinson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 25 17:44:59 PDT 2017


probinson added a comment.

Sorry it took so long to wrap my head around this, but it has been about 20 years since I've had to accommodate an intentional ABI breakage, and that's actually what you want to do.  Remembering that case, I have a model for this one, and I understand what you are doing now.

Be that as it may, `-fclang-abi-compat` is meaningless on PS4, because every time there's a case for checking ClangABICompat, PS4 will want to do it the 3.2 way regardless.  And we'll just make those checks in all those places.  At least it will be relatively easy to audit, and I'm sure our licensees will be happy to ignore the new flag.

So, I am convinced, and the patch LGTM.


Repository:
  rL LLVM

https://reviews.llvm.org/D36501





More information about the cfe-commits mailing list