Typos: test/Interop

[Sajjon]

Fix typos in test/Interop

This is one batch of many PRs fixing typos, see the tracking issue.

[egorzhdan]

@swift-ci please test

[egorzhdan]

@swift-ci please test

[Sajjon]

@egorzhdan @AnthonyLatsis Sorry for downtime, I was on vacation, but back now.

I've squashed and applied changes on top of main ( commit: 906efcb )

Anything I need to do to get this merged?

[egorzhdan]

@swift-ci please test

[AnthonyLatsis]

Anything I need to do to get this merged?

See #75032 (comment).

[AnthonyLatsis]

@swift-ci please smoke test

[AnthonyLatsis]

See the logs. It appears the change in the symbol name also changed the relative order of its lowered definition in the checked output. s9Functions17passThroughCFloatyS2fF now comes after s9Functions18passThroughCDoubleyS2dF and should be checked accordingly in the failed tests.

[Sajjon]

@AnthonyLatsis I've put s9Functions17passThroughCFloatyS2fF directly after s9Functions18passThroughCDoubleyS2dF in these three files, hope what was correct?

(Git diffs vs initial commit, added to the single commit)

[AnthonyLatsis]

Looks good, thank you. Except, did test/Interop/SwiftToC/functions/swift-primitive-functions-execution.c fail as well? I’m pretty sure it was just the other two.

[AnthonyLatsis]

@swift-ci please smoke test Linux

[AnthonyLatsis]

OK, we’re green. If you could kindly exclude just that last change from the middle screenshot, I will wrap this up for you.

[Sajjon]

OK, we’re green. If you could kindly exclude just that last change from the middle screenshot, I will wrap this up for you.

Got it, fixing within the hour.

The CI logs were deleted I think, so I could not see which files needed fixing. Or I just looked in the wrong place.

[Sajjon]

@AnthonyLatsis I've reverted the changes to test/Interop/SwiftToC/functions/swift-primitive-functions-execution.c now :)

[AnthonyLatsis]

I meant just the last change, not a full reversion. The mangled name must be updated, but the permutation was unnecessary. Perhaps I was not clear enough, sorry.

[Sajjon]

I meant just the last change, not a full reversion. The mangled name must be updated, but the permutation was unnecessary. Perhaps I was not clear enough, sorry.

Ofc... silly me. The mangled name change should now be included in test/Interop/SwiftToC/functions/swift-primitive-functions-execution.c, without any permutation of order of lines.

[AnthonyLatsis]

@swift-ci please smoke test

[AnthonyLatsis]

@egorzhdan @zoecarver Is this good to go?

[AnthonyLatsis]

@swift-ci please smoke test