Skip to content

Conversation

@DominikAdamski
Copy link
Contributor

@DominikAdamski DominikAdamski commented Dec 4, 2025

Merge only with: ROCm/llvm-project#757

Motivation

Modify aomp scripts to test PR: ROCm/llvm-project#757

Technical Details

This PR adds required changes to build GPU LLVM IR Flang runtime.

Test Plan

Test procedure:

  1. Apply this PR
  2. Apply LLVM PR: [Flang-RT] Changes required for embedded GPU LLVM IR Flang runtime llvm-project#757 for LLVM
  3. Build AOMP :
./build_aomp.sh

a) (Optional) Build flang_rt.hostdevice if you want to use old -lflang_rt.hostdevice

./build_llvm-flang-rt-host-dev.sh
  1. Test smoke-fort-dev for the new Flang RT:
./check_smoke_fort_dev.sh
  1. (Optional) Test smoke-fort-dev for the old Flang RT (flang_rt.hostdevice):
    Make sure, you built flang_rt.hostdevice (point 3a), and launch smoke_fort_dev tests:
IMPLICIT_GPU_FLANG_RT=0 ./check_smoke_fort_dev.sh

Test Result

No regression for smoke-fort-dev.

Copy link
Contributor

@dpalermo dpalermo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good on 2025-12-10 x-aomp testing

dpalermo pushed a commit to ROCm/llvm-project that referenced this pull request Dec 10, 2025
)

## Motivation

Reduce linking time by embedding GPU LLVM IR code for Flang runtime.

## Technical Details

Modified Flang runtime (added ifdefs to eliminate host-specific code
from GPU compilation).

## Test Plan

Checked smoke-fort-dev tests with new GPU Flang RT and the old
flang_rt.hostdevice.

## Test Result

I was able to compile and successfully run tests without
flang_rt.hostdevice flag together with:
ROCm/aomp#1756
@dpalermo dpalermo merged commit de5a425 into aomp-dev Dec 10, 2025
4 checks passed
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.

3 participants