Skip to content

gfx1200 memory limits milestone3-babel (#1671) - #1698

Merged
geomin12 merged 2 commits into
release/rocm-rel-7.2from
amd/dev/rlieberm/gfx1200-fix
Dec 1, 2025
Merged

gfx1200 memory limits milestone3-babel (#1671)#1698
geomin12 merged 2 commits into
release/rocm-rel-7.2from
amd/dev/rlieberm/gfx1200-fix

Conversation

@ronlieb

@ronlieb ronlieb commented Nov 22, 2025

Copy link
Copy Markdown
Contributor
gfx1200 memory limits milestone3-babel (#1671)

swdev-565113

swdev-565113

 Please enter the commit message for your changes. Lines starting

@carlobertolli carlobertolli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Small nits can be fixed post merge

save_status
./milestone-3-babel -n 20 -s 100000000 # 2400.0 MB
save_status
if [[ "$AOMP_GPU" =~ ^gfx12.* ]]; then

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would limit it to gfx1200 and gfx1201


$PRECMD ./milestone-3-babel -n 20 -s 100000000 # 2400.0 MB
save_status
if [[ "$AOMP_GPU" =~ ^gfx12.* ]]; then

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ditto

@gregrodgers gregrodgers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 29, 2025
@ronlieb ronlieb reopened this Nov 29, 2025
@ronlieb ronlieb closed this Nov 30, 2025
@ronlieb ronlieb reopened this Nov 30, 2025
@ronlieb ronlieb closed this Nov 30, 2025
@ronlieb ronlieb reopened this Nov 30, 2025
@ronlieb ronlieb closed this Nov 30, 2025
@ronlieb ronlieb reopened this Nov 30, 2025
@ronlieb ronlieb closed this Nov 30, 2025
@ronlieb ronlieb reopened this Nov 30, 2025
@rahulc-gh

Copy link
Copy Markdown

re-running test seems infra issue - http://rocm-ci.amd.com/job/rocm-tests/34924/

@geomin12

geomin12 commented Dec 1, 2025

Copy link
Copy Markdown

Landing this as PSDB rocm-ci failures are unrelated, as this is a fortran update. Failures in rocgdb, ocl, rocprof are irrelevant

@geomin12
geomin12 merged commit 740c9bc into release/rocm-rel-7.2 Dec 1, 2025
15 of 16 checks passed
nicebert added a commit that referenced this pull request Mar 10, 2026
Update 27 tests (19 smoke, 8 smoke-limbo) to use replacement flags
for specialized kernel generation. The -fopenmp-target-ignore-env-vars
flag is removed from ROCm/llvm-project (PR #1698) as it had no codegen
implementation and represented a deprecated approach.

Replaced with -fopenmp-assume-teams-oversubscription and
-fopenmp-assume-threads-oversubscription, which provide equivalent
functionality by indicating that the runtime can ignore environment
variables for thread/team configuration, enabling no-loop and
xteam-reduction kernel optimizations.
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.

6 participants