Skip to content

refactor(pathfinder): author candidates in search order - #2708

Draft
rwgk wants to merge 3 commits into
NVIDIA:mainfrom
rwgk:pathfinder_catalog_order_search_order_cleanup
Draft

refactor(pathfinder): author candidates in search order#2708
rwgk wants to merge 3 commits into
NVIDIA:mainfrom
rwgk:pathfinder_catalog_order_search_order_cleanup

Conversation

@rwgk

@rwgk rwgk commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Description

Follow-up to #2689 and the ordering discussion in
#2689 (comment).

Pathfinder historically authored dynamic-library filenames in ascending order,
then reversed them at each runtime lookup site. That distinction no longer
serves the descriptor catalog and makes preference mistakes easier.

This PR:

  • authors Linux SONAMEs and Windows DLL names directly in runtime preference
    order, most preferred first;
  • makes filesystem discovery, already-loaded detection, and native loader
    search consume the descriptor tuples as-is;
  • preserves the historical order of the private legacy SUPPORTED_*
    projections at their compatibility boundary; and
  • documents that ordered candidates for dynamic libraries, headers, binary
    utilities, static libraries, and bitcode libraries are consumed first to
    last.

The tuple rewrite is mechanically behavior-neutral: every multi-name dynamic
library tuple is the exact reverse of its previous catalog representation, and
the corresponding runtime reversal is removed.

Validation

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • Full cuda-pathfinder suite: 1,608 passed, 5 skipped on CUDA 13.3.
  • Repository-wide pre-commit suite (excluding the network-only link
    checker).

@rwgk rwgk added this to the cuda.pathfinder next milestone Aug 27, 2026
@rwgk rwgk added the cuda.pathfinder Everything related to the cuda.pathfinder module label Aug 27, 2026
@copy-pr-bot

copy-pr-bot Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@rwgk rwgk self-assigned this Aug 27, 2026
@rwgk

rwgk commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test e7705a7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cuda.pathfinder Everything related to the cuda.pathfinder module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant