[all-commits] [llvm/llvm-project] cc8f7c: [ORC][LibraryResolver] Fix ensureFilterBuilt asser...

[SahilPatidar via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cc8f7cd2521c98c53f3cdb053b689543c36671c4
      https://github.com/llvm/llvm-project/commit/cc8f7cd2521c98c53f3cdb053b689543c36671c4
  Author: SahilPatidar <sahilpatidar60 at gmail.com>
  Date:   2025-11-06 (Thu, 06 Nov 2025)

  Changed paths:
    M llvm/include/llvm/ExecutionEngine/Orc/TargetProcess/LibraryResolver.h
    M llvm/lib/ExecutionEngine/Orc/TargetProcess/LibraryResolver.cpp
    M llvm/lib/ExecutionEngine/Orc/TargetProcess/LibraryScanner.cpp

  Log Message:
  -----------
  [ORC][LibraryResolver] Fix ensureFilterBuilt assertion failure and concurrency issue. (#166510)

- Fixed architecture compatibility check.
Previously, we used `sys::getDefaultTriple()`, which caused issues on
build bots
using cross-compilation. We now ensure that the target architecture
where the
shared library (.so) is run or loaded matches the architecture it was
built for.

- Fixed ensureFilterBuilt assertion failure.

- Replaced use of FilteredView with a safer alternative for concurrent
environments.
The old FilteredView approach iterated over shared state without
sufficient
synchronization, which could lead to invalid accesses when libraries
were being
added or removed concurrently.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications