Commit 793b46f
committed
ASTMangler: sink fix to be more safe
My previous fix in `6f6a46f` was the correct fix in theory, but in
practice it could accidentally change the mangling of something I
haven't considered, which would break ABI with Swift 6.0
I've narrowed that fix here to only affect dependent conformances
specifically for Copyable/Escapable. The existing code in
`appendDependentProtocolConformance` would always reach a trap because
we're mangling a conformance path that ends with Copyable/Escapable.
We can assume no such symbol has been successfully been mangled before,
thanks to the pre-existing skip in `conformanceRequirementIndex`, so
there's no risk of ABI change.
rdar://1353100191 parent db7a030 commit 793b46f
1 file changed
+13
-5
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4192 | 4192 | | |
4193 | 4193 | | |
4194 | 4194 | | |
4195 | | - | |
4196 | | - | |
4197 | | - | |
4198 | | - | |
4199 | | - | |
| 4195 | + | |
| 4196 | + | |
| 4197 | + | |
| 4198 | + | |
| 4199 | + | |
| 4200 | + | |
| 4201 | + | |
4200 | 4202 | | |
4201 | 4203 | | |
4202 | 4204 | | |
| 4205 | + | |
| 4206 | + | |
| 4207 | + | |
4203 | 4208 | | |
4204 | 4209 | | |
4205 | 4210 | | |
4206 | 4211 | | |
| 4212 | + | |
| 4213 | + | |
| 4214 | + | |
4207 | 4215 | | |
4208 | 4216 | | |
4209 | 4217 | | |
| |||
0 commit comments