rsmith added a comment.

In https://reviews.llvm.org/D36501#852765, @probinson wrote:

> I would prefer a diagnostic if PS4 && >3.2.  Whether you map "latest" to 3.2 
> or diagnose that also, up to you, I'm okay either way.


On reflection, I don't think doing anything special for PS4 here is really 
correct. (Suppose Clang 5 has some (accidental) ABI change in it for PS4, and 
we fix that in Clang 6 to once again follow the psABI and do what Clang 3.2 
did. In that case, building with `-fclang-abi-compat=5` should theoretically 
reinstate that accidental ABI change.) The old-Clang compatibility mode and the 
platform ABI are separate, and we should not conflate them. New patch on the 
way.


Repository:
  rL LLVM

https://reviews.llvm.org/D36501



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to