profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/DougGregor/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

DougGregor/swift-evolution 33

This maintains proposals for changes and user-visible enhancements to the Swift Programming Language.

DougGregor/swift-concurrency-objc 17

Swift Concurrency Translation of Objective-C APIs for iOS, macOS, tvOS, and watchOS

DougGregor/swift 6

The Swift Programming Language

airspeedswift/swift 2

The Swift Programming Language

DougGregor/clang 0

Mirror of official clang git repository located at http://llvm.org/git/clang. Updated every five minutes.

DougGregor/swift-corelibs-libdispatch 0

The libdispatch Project, (a.k.a. Grand Central Dispatch), for concurrency on multicore hardware

DougGregor/swift-corelibs-xctest 0

The XCTest Project, A Swift core library for providing unit test support

pull request commentapple/swift-driver

PrebuiltModuleGen: add support for -module-cache-path

@swift-ci please test

nkcsgexi

comment created time in an hour

Pull request review commentapple/swift

[DNM] Placeholder types: take two

 Expr *ExprRewriter::coerceToType(Expr *expr, Type toType,                                  Optional<Pattern*> typeFromPattern) {   auto &ctx = cs.getASTContext(); +  // Diagnose conversions to invalid function types that couldn't be performed+  // beforehand because of placeholders.+  if (auto *fnTy = toType->getAs<FunctionType>()) {+    auto contextTy = cs.getContextualType(expr);+    if (cs.getConstraintLocator(locator)->isForContextualType() &&+        contextTy && contextTy->hasPlaceholder()) {+      TypeChecker::diagnoseInvalidFunctionType(fnTy, expr->getLoc(), None, dc,+                                               None);

🤞

Jumhyn

comment created time in an hour

pull request commentapple/swift

Add support to skip watchOS 32-bit simulator by default

@swift-ci test macOS

shahmishal

comment created time in an hour

pull request commentapple/swift

Add support to skip watchOS 32-bit simulator by default

@swift-ci smoke test Linux

shahmishal

comment created time in an hour

push eventapple/swift

Mishal Shah

commit sha 3e81252f00961968b9589e0c83e572cb1b4cb303

Disable test to verify watchOS host tests are skipped (76823842)

view details

push time in an hour

Pull request review commentapple/swift

[DNM] Placeholder types: take two

 Expr *ExprRewriter::coerceToType(Expr *expr, Type toType,                                  Optional<Pattern*> typeFromPattern) {   auto &ctx = cs.getASTContext(); +  // Diagnose conversions to invalid function types that couldn't be performed+  // beforehand because of placeholders.+  if (auto *fnTy = toType->getAs<FunctionType>()) {+    auto contextTy = cs.getContextualType(expr);+    if (cs.getConstraintLocator(locator)->isForContextualType() &&+        contextTy && contextTy->hasPlaceholder()) {+      TypeChecker::diagnoseInvalidFunctionType(fnTy, expr->getLoc(), None, dc,+                                               None);

Ok! I’ll give that a shot and see if there’s any unexpected fallout.

Jumhyn

comment created time in 2 hours

pull request commentapple/swift

Add support to skip watchOS 32-bit simulator by default

Build failed Swift Test OS X Platform Git Sha - df28939255e45326189ee956ce9c088665c91f24

shahmishal

comment created time in 3 hours

pull request commentapple/swift

C++ Interop: fix crash for Swift extensions of C++ classes declared in namespaces

@swift-ci please smoke test macOS

egorzhdan

comment created time in 3 hours

Pull request review commentapple/swift

[DNM] Placeholder types: take two

 Expr *ExprRewriter::coerceToType(Expr *expr, Type toType,                                  Optional<Pattern*> typeFromPattern) {   auto &ctx = cs.getASTContext(); +  // Diagnose conversions to invalid function types that couldn't be performed+  // beforehand because of placeholders.+  if (auto *fnTy = toType->getAs<FunctionType>()) {+    auto contextTy = cs.getContextualType(expr);+    if (cs.getConstraintLocator(locator)->isForContextualType() &&+        contextTy && contextTy->hasPlaceholder()) {+      TypeChecker::diagnoseInvalidFunctionType(fnTy, expr->getLoc(), None, dc,+                                               None);

Frankly I'm not sure exactly why it wasn't made to fail, I'd suggest you try to return ErrorType wrapping fnTy for cases when diagnoseInvalidFunctionType would produce an error in resolveASTFunctionType (so that code completion would still have access to underlying type) and return nullptr in coerceToType.

Jumhyn

comment created time in 4 hours

pull request commentapple/swift

Add support to skip watchOS 32-bit simulator by default

00:03:20.860 + ./utils/python_lint.py
00:03:25.777 ./test/attr/Inputs/access-note-gen.py:28:10: E127 continuation line over-indented for visual indent
00:03:25.777 ./test/attr/Inputs/access-note-gen.py:29:10: E127 continuation line over-indented for visual indent

Not related to this PR

shahmishal

comment created time in 4 hours

pull request commentapple/swift

Add support to skip watchOS 32-bit simulator by default

@swift-ci Python lint

shahmishal

comment created time in 5 hours

pull request commentapple/swift

Add support to skip watchOS 32-bit simulator by default

@swift-ci test macOS

shahmishal

comment created time in 5 hours

pull request commentapple/swift

Add support to skip watchOS 32-bit simulator by default

@swift-ci smoke test Linux

shahmishal

comment created time in 5 hours

PR opened apple/swift

Add support to skip watchOS 32-bit simulator by default
+28 -0

0 comment

4 changed files

pr created time in 5 hours

create barnchapple/swift

branch : skip-32bit-watchos-sim

created branch time in 5 hours

pull request commentapple/swift

[DiagnosticQol][SR-14505] Use DeclDescriptive kind in missing return data flow diagnostics

@swift-ci Please smoke test

LucianoPAlmeida

comment created time in 5 hours

push eventapple/swift

Mishal Shah

commit sha 3fd82a182b1bdf33cd0079e28fb62ecbf8ba2273

[5.5] Add support for release/5.5 branch in update-checkout script

view details

Mishal Shah

commit sha 3b3f173ebde6e3b723ce9c81c10ae7b7c881f37b

Merge pull request #36950 from apple/shahmishal/5.5-support-new-branch [5.5] Add support for release/5.5 branch in update-checkout script

view details

push time in 5 hours

delete branch apple/swift

delete branch : shahmishal/5.5-support-new-branch

delete time in 5 hours

pull request commentapple/swift

[silmem2reg] Compute DomTreeLevels lazily.

@swift-ci smoke test

gottesmm

comment created time in 8 hours

PR opened apple/swift

[silmem2reg] Compute DomTreeLevels lazily.

We previously were computing this eagerly meaning that if we did not actually need the DomTreeLevel map, we would calculate it and additionally incur a relatively large malloc for the DenseMap (which stores by default IIRC 64 key value pairs).

Now, we do it lazily ensuring that we only compute this if we actually need it (when promoting non-single block stack allocations).

+39 -28

0 comment

1 changed file

pr created time in 8 hours

push eventapple/swift

Richard Wei

commit sha adb916849d1eb3e8d14b5e611aa757fee8cc1cd7

[AutoDiff] [AST] Handle null decl in 'printDifferentiableAttrArguments'. `DeclAttribute::print()` accepts null decls and has a null default argument. A segfault was triggered in SourceKitService because `printDifferentiableAttrArguments()` took a null decl and deferenced it. This patch makes `printDifferentiableAttrArguments()` handle null decls — if the decl is null, we do not print the differentiability parameters or the `where` clause. I have not added a unit test however because I haven't been able to reproduce it independently. This will be done later as this change is low-risk. Resolves rdar://76095289.

view details

Richard Wei

commit sha f17142883421ec0a240dd7d2208dd4dd79aec612

Merge pull request #36954 from rxwei/76095289

view details

swift_jenkins

commit sha 8a34f2907c7f22a95d7cd83557435a25398e8e12

Merge remote-tracking branch 'origin/main' into next

view details

push time in 9 hours

push eventapple/swift

Richard Wei

commit sha adb916849d1eb3e8d14b5e611aa757fee8cc1cd7

[AutoDiff] [AST] Handle null decl in 'printDifferentiableAttrArguments'. `DeclAttribute::print()` accepts null decls and has a null default argument. A segfault was triggered in SourceKitService because `printDifferentiableAttrArguments()` took a null decl and deferenced it. This patch makes `printDifferentiableAttrArguments()` handle null decls — if the decl is null, we do not print the differentiability parameters or the `where` clause. I have not added a unit test however because I haven't been able to reproduce it independently. This will be done later as this change is low-risk. Resolves rdar://76095289.

view details

Richard Wei

commit sha f17142883421ec0a240dd7d2208dd4dd79aec612

Merge pull request #36954 from rxwei/76095289

view details

push time in 9 hours

PR merged apple/swift

[AutoDiff] [AST] Handle null decl in 'printDifferentiableAttrArguments'.

DeclAttribute::print() accepts null decls and has a null default argument. A segfault was triggered in SourceKitService because printDifferentiableAttrArguments took a null decl and deferenced it. This patch makes printDifferentiableAttrArguments() handle null decls — if the decl is null, we do not print the differentiability parameters or the where clause.

I have not added a unit test however because I haven't been able to reproduce it independently. This will be done later as this change is low-risk.

Resolves rdar://76095289.

+18 -11

2 comments

1 changed file

rxwei

pr closed time in 9 hours

push eventapple/swift

Michael Gottesman

commit sha 90c0bf6e14b3d87e445f07fba63ff5e2c96e066d

[gardening] Remove an unnecessary bounds check on a container. As suggested by @atrick in review feedback in #36913.

view details

Michael Gottesman

commit sha 7345fd8e47f3b5ca6861eb34586d3283d56d6c4a

Merge pull request #36924 from gottesmm/pr-9a40bc50803b002b698f4b755a1f07fdaf6bd02d [gardening] Remove an unnecessary bounds check on a container.

view details

swift_jenkins

commit sha 80a677fde5949ed2c76a698f8ffe6a50bd258856

Merge remote-tracking branch 'origin/main' into next

view details

push time in 9 hours

push eventapple/swift

Michael Gottesman

commit sha 90c0bf6e14b3d87e445f07fba63ff5e2c96e066d

[gardening] Remove an unnecessary bounds check on a container. As suggested by @atrick in review feedback in #36913.

view details

Michael Gottesman

commit sha 7345fd8e47f3b5ca6861eb34586d3283d56d6c4a

Merge pull request #36924 from gottesmm/pr-9a40bc50803b002b698f4b755a1f07fdaf6bd02d [gardening] Remove an unnecessary bounds check on a container.

view details

push time in 10 hours

PR merged apple/swift

[gardening] Remove an unnecessary bounds check on a container.

As suggested by @atrick in review feedback in #36913.

+0 -1

1 comment

1 changed file

gottesmm

pr closed time in 10 hours

pull request commentapple/swift

[DiagnosticQol][SR-14505] Use DeclDescriptive kind in missing return data flow diagnostics

@swift-ci Please smoke test

LucianoPAlmeida

comment created time in 10 hours

pull request commentapple/swift

[DiagnosticQol][SR-14505] Use DeclDescriptive kind in missing return data flow diagnostics

@swift-ci Please smoke test

LucianoPAlmeida

comment created time in 12 hours