[PATCH] D55869: Convert some ObjC retain/release msgSends to runtime calls

2018-12-20 Thread Jonathan Schleifer via Phabricator via cfe-commits
js added a comment. In D55869#1337707 , @dexonsmith wrote: > In D55869#1337692 , @rjmccall wrote: > > > Okay. That's also presumably true for open-source runtimes that support > > ARC; tagging David Chisnall and

[PATCH] D55869: Convert some ObjC retain/release msgSends to runtime calls

2018-12-20 Thread Jonathan Schleifer via Phabricator via cfe-commits
js added a comment. In D55869#1337711 , @dexonsmith wrote: > In D55869#1337706 , @js wrote: > > > The ObjFW runtime itself does not contain anything about release, retain or > > autorelease - these are all part of

[PATCH] D55869: Convert some ObjC retain/release msgSends to runtime calls

2018-12-20 Thread Jonathan Schleifer via Phabricator via cfe-commits
js added a comment. Thanks for tagging me! The ObjFW runtime itself does not contain anything about release, retain or autorelease - these are all part of ObjFW (as in the framework). As ObjFW also supports the Apple runtime, as well as mixing with Foundation code, it so far only provides

[PATCH] D53696: [Haiku] Support __float128 for Haiku x86 and x86_64

2018-11-21 Thread Jonathan Schleifer via Phabricator via cfe-commits
js added inline comments. Comment at: lib/Basic/Targets/OSTargets.h:262 + Builder.defineMacro("__FLOAT128__"); + Builder.defineMacro("_GLIBCXX_USE_FLOAT128"); +} This seems weird. Shouldn't by stdlibc++ and not by the compiler? Repository: rC

[PATCH] D53696: [Haiku] Support __float128 for Haiku x86 and x86_64

2018-11-21 Thread Jonathan Schleifer via Phabricator via cfe-commits
js requested changes to this revision. js added inline comments. This revision now requires changes to proceed. Comment at: lib/Basic/Targets/OSTargets.h:262 + Builder.defineMacro("__FLOAT128__"); + Builder.defineMacro("_GLIBCXX_USE_FLOAT128"); +}

[PATCH] D136089: [clang] Add MorphOS Target

2022-10-17 Thread Jonathan Schleifer via Phabricator via cfe-commits
js created this revision. Herald added subscribers: kbarton, nemanjai. Herald added a project: All. js requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This adds MorphOS to Target and sets up the required defines. Repository: rG LLVM

[PATCH] D136089: [clang] Add MorphOS Target

2022-10-17 Thread Jonathan Schleifer via Phabricator via cfe-commits
js updated this revision to Diff 468226. js added a reviewer: chandlerc. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136089/new/ https://reviews.llvm.org/D136089 Files: clang/lib/Basic/Targets.cpp clang/lib/Basic/Targets/OSTargets.h clang/lib/Basic/Targets/PPC.h