[swift-dev] [Swift CI] Build Failure: 1. OSS - Swift ASAN - OS X (master) #212

[Michael Gottesman]

> On Mar 19, 2017, at 4:59 PM, Greg Parker <gparker at apple.com> wrote:
> 
> Michael, one of your changes broke all three ASAN bots. Please fix or revert ASAP.
> 
> (Could this be related to https://github.com/apple/swift/pull/8169 <https://github.com/apple/swift/pull/8169> ? I didn't confirm that the bot's change lists are correct.)

It is more complicated than you would think. I looked at this earlier. The change itself looks benign and in fact it was happening without ASAN when I had written it in a previous manner. I thought it was a compiler bug so I rewrote it in a simpler manner and the problem went away. If you look at 53eab13be8dd1202428af1b5c315f17c1ab02186, I am just moving some code into a method.

It is not related to the other PR which is due to more COW copies occuring. It appears that I fixed all of the COW issues with a release stdlib (when client is compiled in Debug), but there may be additional COW copies when the stdlib is compiled in Debug mode. = /.

Michael

> 
> 
>> On Mar 18, 2017, at 11:48 AM, no-reply at swift.org <mailto:no-reply at swift.org> wrote:
>> 
>> [FAILURE] oss-swift-incremental-ASAN-RA-osx [#212]
>> 
>> Build URL:	https://ci.swift.org/job/oss-swift-incremental-ASAN-RA-osx/212/ <https://ci.swift.org/job/oss-swift-incremental-ASAN-RA-osx/212/>
>> Project:	oss-swift-incremental-ASAN-RA-osx
>> Date of build:	Sat, 18 Mar 2017 10:26:14 -0700
>> Build duration:	1 hr 22 min
>> Identified problems:
>> 
>> Assertion failure: This build failed because of an assertion failure. Below is a list of all errors in the build log:
>> Indication 1 <https://ci.swift.org//job/oss-swift-incremental-ASAN-RA-osx/212/consoleFull#1684097467c38edd5a-00f9-40a2-abac-1db59f7a90ca>
>> Changes
>> 
>> Commit e52eea2199d8c23ceb82b6bbe85aeb341ccb0fa2 by Michael Gottesman:
>> [silgen] When a formal evaluation scope is in an inout conversion scope,
>> 
>> edit: lib/SILGen/FormalEvaluation.cpp
>> edit: lib/SILGen/FormalEvaluation.h
>> 
>> Commit 53eab13be8dd1202428af1b5c315f17c1ab02186 by Michael Gottesman:
>> [silgen] Move foreignErrorPreparation code in ResultPlanBuilder into a
>> 
>> edit: lib/SILGen/ResultPlan.cpp
>> edit: lib/SILGen/ResultPlan.h
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-dev/attachments/20170319/eea0602e/attachment.html>