Re: [swift-dev] [Swift CI] Build Still Failing: 0. OSS - LLDB Incremental - OS X (master-llvm-swift5-transition) #61

2018-01-17 Thread Vedant Kumar via swift-dev
This didn't send to the list because the quoted section of the email was too long. Once again; I cherry-picked Slava's fix for the failures listed here: a5243443d (HEAD -> llvm-swift5-transition, origin/llvm-swift5-transition) Merge pull request #275 from slavapestov/fix-inout-reconstruction-4.

Re: [swift-dev] DebugScope Verification Error

2018-01-16 Thread Vedant Kumar via swift-dev
+ Adrian and Davide, since they work in this area. Hi Raj, > On Jan 15, 2018, at 4:48 PM, Raj Barik via swift-dev > wrote: > > Thanks Mark. You are right, the following assertion fails > > require(!DS || DS->getParentFunction() == I->getFunction(), > "debug scope of instruction be

Re: [swift-dev] [Swift CI] Build Still Failing: 0. OSS - LLDB Incremental - OS X (master-llvm-swift5-transition) #15

2018-01-12 Thread Vedant Kumar via swift-dev
I'm looking into the lldb failures. vedant > On Jan 11, 2018, at 11:09 PM, swift...@swift.org wrote: > > New issue found! > > [FAILURE] oss-lldb-swift-master-llvm-swift5-transition-incremental-osx [#15] > > Build URL: > https://ci.swift.org/job/oss-lldb-swift-master-llvm-swift5-transition-

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

2018-01-11 Thread Vedant Kumar via swift-dev
Sorry for the noise, I see that you'd already taken care of this with d104a7aec405177384a4cf92a101b99297183fed. vedant > On Jan 11, 2018, at 5:36 PM, Vedant Kumar wrote: > > Hi Slava, > > I think there might be a separate compilation breakage related to: IRGen: > Remove FieldAccess::NonConst

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

2018-01-11 Thread Vedant Kumar via swift-dev
Hi Slava, I think there might be a separate compilation breakage related to: IRGen: Remove FieldAccess::NonConstantIndirect. /Users/buildnode/jenkins/workspace/oss-lldb-swift-master-llvm-swift5-transition-incremental-osx/lldb/source/Symbol/SwiftASTContext.cpp:6891:38: error: no member named 'ma

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

2018-01-04 Thread Vedant Kumar via swift-dev
To follow up on this thread, I reverted #13597 (the profiling refactor) because I couldn't get a clean run of the source compat suite. It looks like there's an issue unrelated to #13597 causing a few failures: https://ci.swift.org/view/Source%20Compatibility/job/swift-master-source-compat-suite/

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

2018-01-03 Thread Vedant Kumar via swift-dev
> On Jan 3, 2018, at 7:46 PM, Slava Pestov wrote: > > > >> On Jan 3, 2018, at 7:36 PM, Vedant Kumar > > wrote: >> >>> >>> On Jan 3, 2018, at 7:34 PM, Mark Lacey >> > wrote: >>> >>> >>> On Jan 3, 2018, at 7:31 PM, Vedant Kumar >>>

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

2018-01-03 Thread Vedant Kumar via swift-dev
> On Jan 3, 2018, at 7:34 PM, Mark Lacey wrote: > > > >> On Jan 3, 2018, at 7:31 PM, Vedant Kumar > > wrote: >> >>> >>> On Jan 3, 2018, at 7:23 PM, Mark Lacey >> > wrote: >>> >>> >>> On Jan 3, 2018, at 7:18 PM, Vedant Kumar >>>

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

2018-01-03 Thread Vedant Kumar via swift-dev
> On Jan 3, 2018, at 7:23 PM, Mark Lacey wrote: > > > >> On Jan 3, 2018, at 7:18 PM, Vedant Kumar > > wrote: >> >> This is a bit surprising, since I wouldn't expect this suite of tests to >> stress the code coverage logic. >> >> How were you able to determine that the

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

2018-01-03 Thread Vedant Kumar via swift-dev
This is a bit surprising, since I wouldn't expect this suite of tests to stress the code coverage logic. How were you able to determine that the #13597 caused the break? I'm having a hard time understanding what the logs mean. For example, I see this failure, but I'm not sure how my change rela

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 16.04 - Long Test (swift 4.1) #126

2017-11-08 Thread Vedant Kumar via swift-dev
Does anyone know what's up with swift-corelibs-libdispatch/dispatch/module.modulemap? [2/25] Generating swiftDispatch.o, swift/Dispatch.swiftmodule, swift/Dispatch.swiftdoc FAILED: src/swiftDispatch.o src/swift/Dispatch.swiftmodule src/swift/Dispatch.swiftdoc cd /home/buildnode/jenkins/worksp

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

2017-11-08 Thread Vedant Kumar via swift-dev
Seems unrelated to my coverage cherry picks: + /Users/buildnode/jenkins/workspace/oss-lldb-swift-4.1-incremental-osx/Ninja-ReleaseAssert/lldb-macosx-x86_64/CustomSwift-Release/lldb-gtest --gtest_output=xml:/Users/buildnode/jenkins/workspace/oss-lldb-swift-4.1-incremental-osx/Ninja-ReleaseAssert

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

2017-10-11 Thread Vedant Kumar via swift-dev
Should be a transient issue. The bots just haven't picked up the paired llvm change. vedant > On Oct 11, 2017, at 12:49 PM, no-re...@swift.org wrote: > > [FAILURE] oss-swift-incremental-RA-linux-ubuntu-16_10 [#1001] > > Build URL: > https://ci.swift.org/job/oss-swift-incremental-RA-linux-u

Re: [swift-dev] [Swift CI] Build Still Failing: 0. OSS - Swift Incremental RA - Ubuntu 14.04 - Long Test (master) #799

2017-09-25 Thread Vedant Kumar via swift-dev
I don't think this is related to my commit -- I've just touched docs and llvm-cov. vedant > On Sep 25, 2017, at 5:51 PM, no-re...@swift.org wrote: > > New issue found! > > [FAILURE] oss-swift-incremental-RA-linux-ubuntu-14_04-long-test [#799] > > Build URL: > https://ci.swift.org/job/oss-

Re: [swift-dev] [Swift CI] Build Still Failing: 0. OSS - Swift Incremental RA - Ubuntu 16.04 (master) #596

2017-09-18 Thread Vedant Kumar via swift-dev
I don't think I have changed this or control it: --install-destdir is required to install products. /home/buildnode/jenkins/workspace/oss-swift-incremental-RA-linux-ubuntu-16_04/swift/utils/build-script: fatal error: command terminated with a non-zero exit status 1, aborting /home/buildnode/jenkin

Re: [swift-dev] [Swift CI] Build Still Failing: 0. OSS - Swift Incremental RA - Ubuntu 16.04 (master) #595

2017-09-18 Thread Vedant Kumar via swift-dev
The sync'd llvm change did not make it into this build. vedant > On Sep 18, 2017, at 5:24 PM, no-re...@swift.org wrote: > > New issue found! > > [FAILURE] oss-swift-incremental-RA-linux-ubuntu-16_04 [#595] > > Build URL: > https://ci.swift.org/job/oss-swift-incremental-RA-linux-ubuntu-16_0

Re: [swift-dev] [Swift CI] Build Still Failing: 0. OSS - Swift Incremental RA - Ubuntu 16.04 (master) #2891

2017-05-01 Thread Vedant Kumar via swift-dev
SwiftPMPackageTests.xctest: /home/buildnode/jenkins/workspace/oss-swift-incremental-RA-linux-ubuntu-16_04/swift/include/swift/Runtime/../../../stdlib/public/SwiftShims/RefCount.h:399: bool swift::RefCountBitsT:: doDecrementStrongExtraRefCount(uint32_t) [refcountIsInline = swift::RefCountInlined

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - Ubuntu 16.04 (swift 3.1) #473

2017-04-11 Thread Vedant Kumar via swift-dev
I see an "Unknown IO error", which looks unrelated to my change: --- Test Case 'TestNSXMLDocument.test_validation_success' started at 19:42:05.659 error : Unknown IO error I/O warning : failed to load external entity " http://www.apple.com/DTDs/PropertyList-1.0.dtd " TestFoundation/TestNSXMLDocum

Re: [swift-dev] [Swift CI] Build Failure: 0. OSS - Swift Incremental RA - OS X (swift 4.0) #71

2017-04-03 Thread Vedant Kumar via swift-dev
This doesn't look related to my changes. + Hugh, since he's at least touched the file. vedant > On Apr 3, 2017, at 4:29 PM, no-re...@swift.org wrote: > > [FAILURE] oss-swift-4.0-incremental-RA-osx [#71] > > Build URL:https://ci.swift.org/job/oss-swift-4.0-incremental-RA-osx/71/ > Project:

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

2017-02-14 Thread Vedant Kumar via swift-dev
Yes, I did. > On Feb 14, 2017, at 5:07 PM, Jordan Rose wrote: > > Did you revert yet? This is blocking PR testing for us. > >> On Feb 14, 2017, at 16:07, Vedant Kumar wrote: >> >> I think I'm just going to revert, because it's taking me a while to get >> swift to build. >> >> vedant >> >>>

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

2017-02-14 Thread Vedant Kumar via swift-dev
I think I'm just going to revert, because it's taking me a while to get swift to build. vedant > On Feb 14, 2017, at 4:07 PM, no-re...@swift.org wrote: > > [FAILURE] oss-swift-incremental-RA-linux-ubuntu-16_10 [#1907] > > Build URL: > https://ci.swift.org/job/oss-swift-incremental-RA-linux

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

2017-02-14 Thread Vedant Kumar via swift-dev
Taking a look. vedant > On Feb 14, 2017, at 3:32 PM, Jordan Rose wrote: > > Looks like yours, Vedant? > > >> On Feb 14, 2017, at 15:22, no-re...@swift.org wrote: >> >> New issue found! >> >> [FAILURE] oss-swift-incremental-RA-osx [#8384] >> >> Build URL: https://ci.swift.org/job/oss-swif

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

2016-12-14 Thread Vedant Kumar via swift-dev
Looks like it was a transient issue :(. #7599 is blue. vedant > On Dec 14, 2016, at 5:40 PM, Slava Pestov wrote: > > Does anyone know what happened here? It doesn’t look like any of the patches > in the blame log could have caused this: > > /Users/buildnode/jenkins/workspace/oss-swift-increme

Re: [swift-dev] Advice on adding PGO support

2016-09-08 Thread Vedant Kumar via swift-dev
Hi Erik, Thanks for your advice! > On Sep 7, 2016, at 12:53 PM, Erik Eckstein wrote: > > Hi Vedant, > > nice work! > >> On Sep 6, 2016, at 12:36 PM, Vedant Kumar via swift-dev >> wrote: >> >> Hi swift-dev, >> >> I've been worki

[swift-dev] Advice on adding PGO support

2016-09-06 Thread Vedant Kumar via swift-dev
Hi swift-dev, I've been working on some patches which add basic support for PGO to swift [1]. What I have so far is just a proof-of-concept. I'd like to get some feedback on the approach I've taken. I've added support for loading profile data, matching up execution counts to the right parts of th

[swift-dev] Adding a singleton to swift

2016-09-01 Thread Vedant Kumar via swift-dev
Hi all, I'm experimenting with swift and would like to add a singleton to the compiler in a way that's visible to SILGen. It seemed that CompilerInstance would be the right place to store it, but I don't see a way to access the CI from SILGen. My first question is: what's the best place to put thi

Re: [swift-dev] Fixing some build issues on master-next

2016-06-27 Thread Vedant Kumar via swift-dev
new coverage encoding API vedant > On Jun 27, 2016, at 10:17 AM, Vedant Kumar via swift-dev > wrote: > > Hi all, > > Just as a heads-up, I've got most of the build errors on master-next sorted > out locally. > > If you need to push something to the branch, plea

[swift-dev] Fixing some build issues on master-next

2016-06-27 Thread Vedant Kumar via swift-dev
Hi all, Just as a heads-up, I've got most of the build errors on master-next sorted out locally. If you need to push something to the branch, please wait a bit so we don't end up duplicating work. I'll ping this thread when the branch is ready. thanks, vedant __

[swift-dev] Support for building without the stdlib

2016-05-12 Thread Vedant Kumar via swift-dev
Hi, I have a build preset which omits "stdlib;sdk-overlay" from swift-install-components. After 6670bb76, this preset triggers a swift cmake failure (Unknown build type). Is it no longer allowed to build swift without the stdlib? thanks vedant ___ sw

Re: [swift-dev] Building swift-llvm fails when using build-script to cross compile due to multiple -isysroot arguments being in CFLAGS

2016-03-22 Thread Vedant Kumar via swift-dev
+ swift-dev, Argyrios > On Mar 21, 2016, at 8:06 PM, Andrew Hyatt wrote: > > swift-dev, > > 5e342aa fixed the CFLAGS issue for swift-llvm and it now compiles fine for > iOS. > > A remnant of the issue persists while building Swift. To get to that point I > copied over two headers not prese

Re: [swift-dev] Building swift-llvm fails when using build-script to cross compile due to multiple -isysroot arguments being in CFLAGS

2016-03-19 Thread Vedant Kumar via swift-dev
> On Mar 19, 2016, at 12:05 PM, ChrisBieneman wrote: > > [Adding Vedant directly] > > I'm not super familiar with the swift build scripts, but I have a theory of > what is going wrong. I think the problem is that compiler-rt is being built > during the cross build, and it really shouldn't be.