[llvm] Reapply "[ORC] Replace ORC's baked-in dependence ... (#163027)" with … (PR #164340)
    Jordan Rupprecht via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Oct 24 18:55:11 PDT 2025
    
    
  
rupprecht wrote:
> > WDYT?
> 
> We don't want that -- query handlers need to be run outside the session lock.
I didn't think so, but it was worth a shot ;)
> 
> I think we'll just need to filter the `EmitQueries->Updated` list by `Q->IsComplete()` at the end of `IL_emit` just before we return. So that member should become `EmitQueries::CompletedQueries`, and the loop outside can become:
> 
> ```c++
> for (auto &CQ : EmitQueries->CompletedQueries)
>   CQ->handleComplete(*this);
> ```
Created #165063 to do that, and it fixes the tsan issues.
https://github.com/llvm/llvm-project/pull/164340
    
    
More information about the llvm-commits
mailing list