Skip to content

Conversation

@KushalP
Copy link
Contributor

@KushalP KushalP commented Oct 23, 2025

The issue was that in three tests, the Args arrays were being declared inside for loops.

When TQ.addTask() was called, it stored a pointer to the Args array. As the Args array was declared in the loop, it would go out of scope at the end of each iteration whilst TaskQueue held a pointer to it.

The issue was that in three tests, the `Args` arrays were being declared inside
for loops.

When `TQ.addTask()` was called, it stored a pointer to the `Args` array. As the
`Args` array was declared in the loop, it would go out of scope at the end of
each iteration whilst `TaskQueue` held a pointer to it.
@KushalP
Copy link
Contributor Author

KushalP commented Oct 23, 2025

@swift-ci Please test

@KushalP
Copy link
Contributor Author

KushalP commented Oct 23, 2025

preset=asan
@swift-ci Please test with preset macOS platform

1 similar comment
@KushalP
Copy link
Contributor Author

KushalP commented Oct 23, 2025

preset=asan
@swift-ci Please test with preset macOS platform

@KushalP
Copy link
Contributor Author

KushalP commented Oct 23, 2025

@swift-ci Please test macOS platform

@KushalP
Copy link
Contributor Author

KushalP commented Oct 24, 2025

preset=asan
@swift-ci Please test with preset macOS platform

@glessard
Copy link
Contributor

glessard commented Oct 24, 2025

The test with preset=asan has timed out twice now. Either there is something wrong with the configuration of the bot "swift-PR-osx-preset" that makes it fail with the ASAN preset, or it is an error that happens with ASAN enabled. I am inclined to merge and let the regular ASAN bot find whether it is the second case.

@glessard glessard merged commit d4a9b3f into swiftlang:main Oct 24, 2025
5 of 6 checks passed
@KushalP KushalP deleted the stack-use-after-scope-TaskQueueTest branch October 24, 2025 18:18
@AnthonyLatsis
Copy link
Collaborator

@KushalP For the future, you can grab the preset from the logs of a job. For example, https://ci.swift.org/job/oss-swift-incremental-ASAN-RA-macos is using buildbot_incremental_asan,tools=RDA,stdlib=RDA.

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.

4 participants