Re: [swift-dev] Swift Incremental broken

2017-11-29 Thread Arnold Schwaighofer via swift-dev
Looks like this is caused by https://github.com/apple/swift/pull/12952 . Reverting here: https://github.com/apple/swift/pull/13144 > On Nov 28, 2017, at 9:30 PM, Arnold Schwaighofer via swift-dev > wrote: > > The incremental OS X bot is broken. > >

[swift-dev] Protocol Devirtualizer Pass

2017-11-29 Thread Raj Barik via swift-dev
Hi, I am thinking about writing a Protocol Devirtualizer Pass that specializes functions that take Protocols as arguments to transform them with concrete types instead of protocol types when the concrete types can be determined statically by some compiler analysis. This is the first step of the tr

Re: [swift-dev] Protocol Devirtualizer Pass

2017-11-29 Thread Arnold Schwaighofer via swift-dev
The issue I raised is when we truly replace the type in an existing function: protocol Proto :class {} class SingleImpl : Proto {} class User { func useProto(p: Proto) {} } If i understand you correctly your pass changes this program to: class User { func useProto(p: SingleImpl) {} }

Re: [swift-dev] Protocol Devirtualizer Pass

2017-11-29 Thread Slava Pestov via swift-dev
Hi Raj, The way I would approach this problem is first, turn a function taking a protocol value into one taking a protocol-constrained generic parameter. So @inline(never) internal func wrap_inc(a:SumProtocol, val:Int) -> Int{ return a.increment(i:val) } Would become @inline(always) internal

[swift-dev] Libc portability, _GNU_SOURCE, platform checks... oh my

2017-11-29 Thread Zac Bowling via swift-dev
Hi everyone, I've been working through some issues around figuring out the proper way of handling portable Swift code, specifically around libc, as part of adding support for Fuchsia to swift. Some of this has been discussed on this list in the past but I wanted to start a new thread to discuss s

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 16.10 (master) #1746

2017-11-29 Thread Arnold via swift-dev
%{ This seems one of those missing dependency errors that keeps popping up ... usually clears up with a clean or later build. > On Nov 29, 2017, at 5:56 PM, no-re...@swift.org wrote: > > [FAILURE] oss-swift-incremental-RA-linux-ubuntu-16_10 [#1746] > > Build URL: > https://ci.swift.org/job/

[swift-dev] [Swift CI] Build Still Failing: 0. OSS - LLDB Incremental - OS X (master) #454

2017-11-29 Thread no-reply--- via swift-dev
New issue found!Title: Report [FAILURE] oss-lldb-incremental-osx [#454] Build URL:https://ci.swift.org/job/oss-lldb-incremental-osx/454/ Project:oss-lldb-incremental-osx Date of build:Wed, 29 Nov 2017 22:40:25 -0600 Build duration:1 hr 0 min Identified problems:Regression