-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Closed
Labels
compiler:codegenGeneration of LLVM IR and native codeGeneration of LLVM IR and native codemultithreadingBase.Threads and related functionalityBase.Threads and related functionalityregressionRegression in behavior compared to a previous versionRegression in behavior compared to a previous version
Description
The changes in #53219 appears to have introduced a datarace on the mi->inInference field, which seems likely to lead to incorrect compilation and bad performance of threaded code. It was previously protected by the codegen lock.
Originally posted by @vtjnash in #53219 (comment)
Posting as an issue since the PR is already merged, and this is a regression in thread-safety
Metadata
Metadata
Assignees
Labels
compiler:codegenGeneration of LLVM IR and native codeGeneration of LLVM IR and native codemultithreadingBase.Threads and related functionalityBase.Threads and related functionalityregressionRegression in behavior compared to a previous versionRegression in behavior compared to a previous version