Skip to content

[mandatory-inline] Extract out tryDevirtualizeCode into its own helper function. NFC. #11839

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

gottesmm
Copy link
Contributor

@gottesmm gottesmm commented Sep 9, 2017

[mandatory-inline] Extract out tryDevirtualizeCode into its own helper function. NFC.

Just trying to slim down/refactor this huge loop.

rdar://31521023

@gottesmm
Copy link
Contributor Author

gottesmm commented Sep 9, 2017

@swift-ci smoke test and merge

@gottesmm
Copy link
Contributor Author

gottesmm commented Sep 9, 2017

Actually this is wrong. I need to fix this. brb.

@gottesmm
Copy link
Contributor Author

gottesmm commented Sep 9, 2017

I just killed the job.

@gottesmm gottesmm force-pushed the pr-8aae8140c859cc0621e37aa723e638253cf585b7 branch from 290fead to ca2c91b Compare September 9, 2017 23:19
@gottesmm
Copy link
Contributor Author

gottesmm commented Sep 9, 2017

Ok. Now lets try this again...

@gottesmm
Copy link
Contributor Author

gottesmm commented Sep 9, 2017

@swift-ci smoke test and merge

1 similar comment
@gottesmm
Copy link
Contributor Author

gottesmm commented Sep 9, 2017

@swift-ci smoke test and merge

@gottesmm
Copy link
Contributor Author

sigh need to use make_tuple

…r function. NFC.

Just trying to slim down/refactor this huge loop.

rdar://31521023
@gottesmm gottesmm force-pushed the pr-8aae8140c859cc0621e37aa723e638253cf585b7 branch from ca2c91b to 600edc3 Compare September 10, 2017 02:39
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test and merge

2 similar comments
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test and merge

@gottesmm
Copy link
Contributor Author

@swift-ci smoke test and merge

@swift-ci swift-ci merged commit cf27ab9 into swiftlang:master Sep 10, 2017
@gottesmm gottesmm deleted the pr-8aae8140c859cc0621e37aa723e638253cf585b7 branch September 10, 2017 03:07
ktoso added a commit to ktoso/swift that referenced this pull request May 1, 2025
We missed to sign the handler. Along the way the signature of it
changed, so adjust for that.

How to get the number:

```
func PROPER(bar: (TaskPriority, TaskPriority) -> Void) {
    let p = TaskPriority.default
    bar(p, p)
}
```

```
-> % swiftc -target arm64e-apple-macos13 example.swift -S -o - | swift demangle | grep -a3 autda
	stur	x8, [x29, #-64]
	mov	x17, x8
	movk	x17, swiftlang#11839, lsl swiftlang#48 <<<<<<<<<
	autda	x16, x17
	ldr	x8, [x16, swiftlang#64]
	lsr	x8, x8, #0
	add	x8, x8, swiftlang#15
```

Resolves rdar://150378890
ktoso added a commit to ktoso/swift that referenced this pull request May 1, 2025
We missed to sign the handler. Along the way the signature of it
changed, so adjust for that.

How to get the number:

```
func PROPER(bar: (TaskPriority, TaskPriority) -> Void) {
    let p = TaskPriority.default
    bar(p, p)
}
```

```
-> % swiftc -target arm64e-apple-macos13 example.swift -S -o - | swift demangle | grep -a3 autda
	stur	x8, [x29, #-64]
	mov	x17, x8
	movk	x17, swiftlang#11839, lsl swiftlang#48 <<<<<<<<<
	autda	x16, x17
	ldr	x8, [x16, swiftlang#64]
	lsr	x8, x8, #0
	add	x8, x8, swiftlang#15
```

Resolves rdar://150378890
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