[all-commits] [llvm/llvm-project] 9bf0c4: [orc-rt] Simplify ConnectorRegistry::connect and o...

[Lang Hames via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9bf0c497dceba0c7eb882c0f2a96c8151ce74ae3
      https://github.com/llvm/llvm-project/commit/9bf0c497dceba0c7eb882c0f2a96c8151ce74ae3
  Author: Lang Hames <lhames at gmail.com>
  Date:   2026-09-27 (Sun, 27 Sep 2026)

  Changed paths:
    M orc-rt/include/orc-rt/bedrock/ConnectorRegistry.h
    M orc-rt/lib/bedrock/ConnectorRegistry.cpp
    M orc-rt/lib/bedrock/sys/posix/SocketConnector.cpp
    M orc-rt/test/unit/bedrock/sys/posix/SocketConnectorTest.cpp
    M orc-rt/tools/ogre/ogre.cpp

  Log Message:
  -----------
  [orc-rt] Simplify ConnectorRegistry::connect and ogre setup. (#226772)

ConnectorRegistry::connect and ConnectorFn now take the Session and
BootstrapInfo directly, replacing the GetAttachInfo callback and the
AttachInfo struct. The callback let the Session be built lazily, after
the connector had validated its spec, but a connection can fail after
validation anyway, so callers already had to handle discarding a Session
built for a failed connection. Build the Session first, then connect.

ogre's setup is restructured to match: makeSession detects the process
info, builds a thread-pool dispatcher and the Session, adds the host
services, then connects; runOgre waits for the detach. Errors are
reported via Session::logErrors when logging is enabled.

SocketConnectorTest now connects a real Session. Its ownership test
adopts a non-stream socket, which the connector accepts and
createSimpleRemoteCAOverSocket then rejects, rather than relying on a
failing GetAttachInfo.



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