Skip to content

Enable better runtime failure messages#29903

Merged
eeckstein merged 1 commit into
swiftlang:masterfrom
eeckstein:enable-runtime-messages
Feb 18, 2020
Merged

Enable better runtime failure messages#29903
eeckstein merged 1 commit into
swiftlang:masterfrom
eeckstein:enable-runtime-messages

Conversation

@eeckstein
Copy link
Copy Markdown
Contributor

@eeckstein eeckstein commented Feb 18, 2020

The implementation was done quite a while ago.
Now, that we have support in lldb (swiftlang/llvm-project#773), we can enable it by default in the compiler.

LLDB now shows the runtime failure reason, for example:

* thread #1, queue = 'com.apple.main-thread', stop reason = Swift runtime failure: arithmetic overflow
    frame #1: 0x0000000100000f0d a.out`testit(a=127) at trap_message.swift:4
   1   	
   2   	@inline(never)
   3   	func testit(_ a: Int8) -> Int8 {
-> 4   	  return a + 1
   5   	}
   6   	

For details, see #25978

rdar://problem/51278690

@eeckstein
Copy link
Copy Markdown
Contributor Author

@swift-ci test

The implementation was done quite a while ago.
Now, that we have support in lldb (swiftlang/llvm-project#773), we can enable it by default in the compiler.

LLDB now shows the runtime failure reason, for example:

* thread #1, queue = 'com.apple.main-thread', stop reason = Swift runtime failure: arithmetic overflow
    frame #1: 0x0000000100000f0d a.out`testit(a=127) at trap_message.swift:4
   1
   2   	@inline(never)
   3   	func testit(_ a: Int8) -> Int8 {
-> 4   	  return a + 1
   5   	}
   6

For details, see swiftlang#25978

rdar://problem/51278690
@eeckstein eeckstein force-pushed the enable-runtime-messages branch from 904fe55 to 56ecb8f Compare February 18, 2020 11:04
@eeckstein
Copy link
Copy Markdown
Contributor Author

@swift-ci test

@swift-ci
Copy link
Copy Markdown
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 904fe55385e0690745b902cbf055f9979c3d6ea8

@eeckstein
Copy link
Copy Markdown
Contributor Author

@swift-ci test

@swift-ci
Copy link
Copy Markdown
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 56ecb8f

@eeckstein eeckstein merged commit 60565d5 into swiftlang:master Feb 18, 2020
@eeckstein eeckstein deleted the enable-runtime-messages branch February 18, 2020 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants