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

2017-12-13 Thread David Hart via swift-dev
This seems unrelated to my commit as its a CLF test that is failing: TestURLSession.test_simpleUploadWithDelegate : XCTAssertEqual failed: ("0") is not equal to ("16384") - > On 13 Dec 2017, at 16:25, swift...@swift.org wrote: > > [FAILURE] oss-swift-incremental-RA-linux-ubuntu-16_10 [#1980] >

Re: [swift-dev] [Swift CI] Build Failure: OSS - Swift Package - Ubuntu 16.04 (swift 4.1) #336

2017-12-13 Thread David Hart via swift-dev
mkdir: cannot create directory `/www-data/tmp-builds/swift-4.1-DEVELOPMENT-SNAPSHOT-2017-12-13-a-338': No space left on device 😁 > On 13 Dec 2017, at 17:22, swift...@swift.org wrote: > > [FAILURE] oss-swift-4.1-package-linux-ubuntu-16_04 [#336] > > Build URL: > https://ci.swift.org/job/os

[swift-dev] [Swift CI] Build Failure: OSS - Swift Package - OS X (master) #840

2017-12-13 Thread swift-ci--- via swift-dev
Title: Report [FAILURE] oss-swift-package-osx [#840] Build URL:https://ci.swift.org/job/oss-swift-package-osx/840/ Project:oss-swift-package-osx Date of build:Wed, 13 Dec 2017 06:31:03 -0600 Build duration:2 hr 44 min Changes No Changes __

[swift-dev] [Swift CI] Build Failure: OSS - Swift Package - OS X (swift 4.1) #336

2017-12-13 Thread swift-ci--- via swift-dev
Title: Report [FAILURE] oss-swift-4.1-package-osx [#336] Build URL:https://ci.swift.org/job/oss-swift-4.1-package-osx/336/ Project:oss-swift-4.1-package-osx Date of build:Wed, 13 Dec 2017 06:30:29 -0600 Build duration:2 hr 44 min Changes No Chang

[swift-dev] Disk full ---> Re: [Swift CI] Build Failure: OSS - Swift Package - OS X (master) #840

2017-12-13 Thread David Zarzycki via swift-dev
> On Dec 13, 2017, at 11:34, swift-ci--- via swift-dev > wrote: > > [FAILURE] oss-swift-package-osx [#840] > > Build URL:https://ci.swift.org/job/oss-swift-package-osx/840/ > > Project: oss-swift-package-osx > Date of build:

[swift-dev] Switching swift to C++14

2017-12-13 Thread Saleem Abdulrasool via swift-dev
Hi, The newer Windows SDK requires the use of C++14 (the SDK headers use `auto` return types without trailing type information). Joe mentioned that there was some interest in switching the rest of swift to C++14 as well. I figured that I would just start a thread here to determine if this is oka

Re: [swift-dev] Disk full ---> Re: [Swift CI] Build Failure: OSS - Swift Package - OS X (master) #840

2017-12-13 Thread Mishal Shah via swift-dev
Fixed. > On Dec 13, 2017, at 9:50 AM, David Zarzycki via swift-dev > wrote: > > > > >> On Dec 13, 2017, at 11:34, swift-ci--- via swift-dev > > wrote: >> >> [FAILURE] oss-swift-package-osx [#840] >> >> Build URL: https://ci.swift.org/job/oss-swift-package-osx/

Re: [swift-dev] [Swift CI] Build Failure: OSS - Swift Package - Ubuntu 16.04 (swift 4.1) #336

2017-12-13 Thread Mishal Shah via swift-dev
Fixed > On Dec 13, 2017, at 8:24 AM, David Hart wrote: > > mkdir: cannot create directory > `/www-data/tmp-builds/swift-4.1-DEVELOPMENT-SNAPSHOT-2017-12-13-a-338': No > space left on device > > 😁 > >> On 13 Dec 2017, at 17:22, swift...@swift.org >> wrote: >> >>

Re: [swift-dev] Protocol Devirtualizer Pass

2017-12-13 Thread Raj Barik via swift-dev
Slava, I have two (clarification) questions in your proposed implementation: *Original Function:* @inline(never) internal func wrap_inc(a:SumProtocol, val:Int) -> Int{ return a.increment(i:val) } *Transformed code:* @inline(always) internal func wrap_inc(a: SumProtocol, val: Int) -> Int { // o

Re: [swift-dev] Switching swift to C++14

2017-12-13 Thread Jordan Rose via swift-dev
No one else has commented on this yet today, so I'll put in that I don't have any objections to this and don't foresee any major problems. The one place where we'd need to be careful is with LLDB, which imports Swift headers; if Swift is going to move to C++14, then Swift-LLDB probably has to as

Re: [swift-dev] Switching swift to C++14

2017-12-13 Thread Saleem Abdulrasool via swift-dev
> On Dec 13, 2017, at 1:45 PM, Jordan Rose wrote: > > No one else has commented on this yet today, so I'll put in that I don't have > any objections to this and don't foresee any major problems. The one place > where we'd need to be careful is with LLDB, which imports Swift headers; if > Swift

Re: [swift-dev] Switching swift to C++14

2017-12-13 Thread Jordan Rose via swift-dev
> On Dec 13, 2017, at 14:19, Saleem Abdulrasool wrote: > >> On Dec 13, 2017, at 1:45 PM, Jordan Rose wrote: >> >> No one else has commented on this yet today, so I'll put in that I don't >> have any objections to this and don't foresee any major problems. The one >> place where we'd need to

Re: [swift-dev] Switching swift to C++14

2017-12-13 Thread Saleem Abdulrasool via swift-dev
> On Dec 13, 2017, at 2:30 PM, Jordan Rose wrote: > > > >> On Dec 13, 2017, at 14:19, Saleem Abdulrasool wrote: >> >>> On Dec 13, 2017, at 1:45 PM, Jordan Rose wrote: >>> >>> No one else has commented on this yet today, so I'll put in that I don't >>> have any objections to this and don'

Re: [swift-dev] [swift-lldb-dev] Switching swift to C++14

2017-12-13 Thread Daniel Dunbar via swift-dev
FWIW, llbuild requires C++14. We have to do some minor shenanigans to workaround bugs in libstdc++ on 14.04, and our use is probably minimal, but just throwing that out there. - Daniel > On Dec 13, 2017, at 1:45 PM, Jordan Rose via swift-lldb-dev > wrote: > > No one else has commented on th

Re: [swift-dev] Protocol Devirtualizer Pass

2017-12-13 Thread Arnold Schwaighofer via swift-dev
You don’t need a second open_existential_ref in the _wrap_inc function. It should look something like this: sil @_wrap_inc : $@convention(thin) (@owned T, Int) -> Int { bb0(%0 : $T, %1 : $Int): %5 = witness_method $T, #SumProtocol.inc!1 : (Self) -> (Int) -> Int : $@convention(witness_method

Re: [swift-dev] Preserving and Applying CC in Imported Decls

2017-12-13 Thread Saleem Abdulrasool via swift-dev
> On Dec 13, 2017, at 12:46 PM, John McCall wrote: > >> >> On Dec 13, 2017, at 3:22 PM, Saleem Abdulrasool > > wrote: >> >> >> >>> On Dec 13, 2017, at 12:14 PM, John McCall >> > wrote: >>> On Dec 13, 2017, at 2:56 PM, Sale

Re: [swift-dev] Preserving and Applying CC in Imported Decls

2017-12-13 Thread John McCall via swift-dev
> On Dec 13, 2017, at 6:42 PM, Saleem Abdulrasool wrote: > > > >> On Dec 13, 2017, at 12:46 PM, John McCall > > wrote: >> >>> >>> On Dec 13, 2017, at 3:22 PM, Saleem Abdulrasool >> > wrote: >>> >>> >>> On Dec 13, 2017, at 12:14

Re: [swift-dev] Preserving and Applying CC in Imported Decls

2017-12-13 Thread Saleem Abdulrasool via swift-dev
On Wed, Dec 13, 2017 at 4:14 PM, John McCall wrote: > > On Dec 13, 2017, at 6:42 PM, Saleem Abdulrasool > wrote: > > > > On Dec 13, 2017, at 12:46 PM, John McCall wrote: > > > On Dec 13, 2017, at 3:22 PM, Saleem Abdulrasool > wrote: > > > > On Dec 13, 2017, at 12:14 PM, John McCall wrote: > >

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

2017-12-13 Thread swift-ci--- via swift-dev
Title: Report [FAILURE] oss-lldb-incremental-osx [#506] Build URL:https://ci.swift.org/job/oss-lldb-incremental-osx/506/ Project:oss-lldb-incremental-osx Date of build:Wed, 13 Dec 2017 19:17:30 -0600 Build duration:44 min Identified problems:Regression test failed: This b

Re: [swift-dev] [Swift CI] Build Failure: 2. Swift Source Compatibility Suite (master) #916

2017-12-13 Thread Douglas Gregor via swift-dev
Arnold, these backtraces imply that it’s your change. Sent from my iPhone > On Dec 13, 2017, at 9:54 PM, swift...@swift.org wrote: > > [FAILURE] swift-master-source-compat-suite [#916] > > Build URL:https://ci.swift.org/job/swift-master-source-compat-suite/916/ > Project: swift-master

Re: [swift-dev] Preserving and Applying CC in Imported Decls

2017-12-13 Thread John McCall via swift-dev
> On Dec 13, 2017, at 8:35 PM, Saleem Abdulrasool wrote: > > On Wed, Dec 13, 2017 at 4:14 PM, John McCall > wrote: > >> On Dec 13, 2017, at 6:42 PM, Saleem Abdulrasool > > wrote: >> >> >> >>> On Dec 13, 2017, at 12:46 PM, John McCall

[swift-dev] Make offset index available for String

2017-12-13 Thread Cao, Jiannan via swift-dev
Hi, I would like to discuss the String.Index problem within Swift. I know the current situation of String.Index is based on the nature of the underlaying data structure of the string. But could we just make String.Index contain offset information? Or make offset index subscript available for a

[swift-dev] Adding a CHECK to test case.

2017-12-13 Thread Atul Sowani via swift-dev
Hi, clang_inline_opt.swift test case is failing on ppc64le with following error: /root/swift-source/swift/test/IRGen/clang_inline_opt.swift:10:11: error: expected string not found in input // CHECK: define internal i32 @return7() [[CLANG_ATTRS:#[0-9]+]] { ^ :1:1: note: scanning from her

Re: [swift-dev] [Swift CI] Build Failure: 2. Swift Source Compatibility Suite (master) #916

2017-12-13 Thread Douglas Gregor via swift-dev
I’m checking that theory with https://github.com/apple/swift/pull/13427 and will complete the revert if that PR runs clean. > On Dec 13, 2017, at 10:08 PM, Douglas Gregor via swift-dev > wrote: > > Arnold, these backtraces imply that it’s your change. > > Sent from my iPhone > > On