Skip to content

[lapack][rocSOLVER] Implement gerqf and getri - #782

Open
zjin-lcf wants to merge 2 commits into
uxlfoundation:developfrom
zjin-lcf:feature/rocsolver-gerqf-getri
Open

zjin-lcf wants to merge 2 commits into
uxlfoundation:developfrom
zjin-lcf:feature/rocsolver-gerqf-getri

Conversation

@zjin-lcf

Copy link
Copy Markdown
Contributor

Summary

  • implement non-batched gerqf and getri for buffer and USM APIs
  • retain checked 32-bit pivot conversion for getri, which has no 64-bit rocSOLVER entry point
  • make pivot-conversion, solve, and temporary-storage cleanup ordering explicit
  • add the corresponding scratchpad queries

Extracted from #767 for focused review. This branch is stacked on #778; until that PR merges, it intentionally includes the shared foundation commit.

Test plan

  • clang-format 19.1 check
  • git diff --check
  • rocSOLVER backend and RT/CT test binaries build for gfx90a
  • focused gerqf tests on MI210 with ONEAPI_DEVICE_SELECTOR=hip:*: 24 passed across RT/CT
  • local getri execution is blocked by an assertion in the custom DPC++ runtime's device-kernel registration before the rocSOLVER call; the same assertion reproduces in the existing getrs pivot-conversion path, and CI should exercise this path with the supported toolchain

zjin-lcf and others added 2 commits September 18, 2026 15:41
Add type-generic devInfo readers for legacy and 64-bit rocSOLVER entry points and report all failing matrices through lapack::batch_error.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@zjin-lcf
zjin-lcf requested a review from a team as a code owner September 18, 2026 23:10

This branch has not been deployed

No deployments
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.

1 participant