Skip to content

Fix sycl-time-trace-actual link-mode failure in Self build / Build + LIT#22050

Closed
Copilot wants to merge 2 commits into
syclfrom
copilot/fix-self-build-job
Closed

Fix sycl-time-trace-actual link-mode failure in Self build / Build + LIT#22050
Copilot wants to merge 2 commits into
syclfrom
copilot/fix-self-build-job

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 19, 2026

The failing job was caused by a single Clang driver test: Clang :: Driver/sycl-time-trace-actual.cpp.
Its compile+link SYCL path (--offload-new-driver) emitted wrapper calls to __sycl_register_lib / __sycl_unregister_lib, but the test links without SYCL runtime and failed with undefined references.

  • Root cause

    • Test 3 in sycl-time-trace-actual.cpp performs a real link step.
    • The offload wrapper object now expects SYCL registration symbols during link, which are not available in this test environment.
  • Targeted fix

    • Added local C-linkage stubs for the two SYCL registration hooks directly in the test source so link-mode succeeds without pulling in runtime libraries.
    • Kept the rest of the test logic and expectations unchanged (trace file generation/naming behavior remains the same).
  • Patch excerpt

    extern "C" void __sycl_register_lib(void *) {}
    extern "C" void __sycl_unregister_lib(void *) {}
    
    int main() {
      return 0;
    }

Agent-Logs-Url: https://github.com/intel/llvm/sessions/c44f92b0-7275-4106-9e4d-09e280b9725e

Co-authored-by: srividya-sundaram <8020274+srividya-sundaram@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job self build Fix sycl-time-trace-actual link-mode failure in Self build / Build + LIT May 19, 2026
Copilot AI requested a review from srividya-sundaram May 19, 2026 02:13
@srividya-sundaram srividya-sundaram marked this pull request as ready for review May 19, 2026 18:14
@srividya-sundaram srividya-sundaram requested a review from a team as a code owner May 19, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants