[PATCH] D70575: [Clang] Define Fuchsia C++ABI

2019-12-03 Thread Petr Hosek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG9c3f9b9c12b0: [Clang] Define Fuchsia C++ABI (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D70575?vs=230827&id=232023#toc Repository: rG LLVM Github Monorepo CHANGES SINCE L

[PATCH] D70575: [Clang] Define Fuchsia C++ABI

2019-11-24 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In D70575#1756072 , @leonardchan wrote: > Could you add a test to show that with a fuchsia target we end up returning > `this` from constructors + destructors and ensure that this ABI is used? Done Repository: rG LLVM Github

[PATCH] D70575: [Clang] Define Fuchsia C++ABI

2019-11-24 Thread Petr Hosek via Phabricator via cfe-commits
phosek updated this revision to Diff 230827. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70575/new/ https://reviews.llvm.org/D70575 Files: clang/include/clang/Basic/TargetCXXABI.h clang/lib/AST/ASTContext.cpp clang/lib/Basic/Targets/OSTarget

[PATCH] D70575: [Clang] Define Fuchsia C++ABI

2019-11-21 Thread Leonard Chan via Phabricator via cfe-commits
leonardchan added a comment. Could you add a test to show that with a fuchsia target we end up returning `this` from constructors + destructors and ensure that this ABI is used? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70575/new/ https://revi

[PATCH] D70575: [Clang] Define Fuchsia C++ABI

2019-11-21 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. phosek added a reviewer: leonardchan. Herald added a project: clang. Herald added a subscriber: cfe-commits. It is a modified version of the Itanium ABI. The only change is that constructors and destructors return 'this'. Repository: rG LLVM Github Monorepo https