Skip to content

[Comgr] Log message on temp file creation failure suggesting bad TMPDIR#2465

Open
schung-amd wants to merge 2 commits into
amd-stagingfrom
users/schung-amd/comgr-tmpdir-log
Open

[Comgr] Log message on temp file creation failure suggesting bad TMPDIR#2465
schung-amd wants to merge 2 commits into
amd-stagingfrom
users/schung-amd/comgr-tmpdir-log

Conversation

@schung-amd
Copy link
Copy Markdown

@schung-amd schung-amd commented May 8, 2026

Addresses ROCm/TheRock#3103. Currently when TMPDIR is set to a path that does not exist or is otherwise inaccessible, comgr will fail to create temporary files and throw an error, but the root cause is not suggested or captured anywhere.

The added logging indicates to users that they should check their TMPDIR and will show up in the comgr logs and console output with AMD_LOG_LEVEL=1 and above when AMD_COMGR_EMIT_VERBOSE_LOGS=1.

Comment thread amd/comgr/src/comgr-compiler.cpp Outdated
Comment thread amd/comgr/src/comgr-compiler.cpp Outdated
Comment thread amd/comgr/src/comgr-compiler.cpp Outdated
@z1-cciauto
Copy link
Copy Markdown
Collaborator

@chinmaydd chinmaydd changed the title Log message on temp file creation failure suggesting bad TMPDIR [Comgr] Log message on temp file creation failure suggesting bad TMPDIR May 8, 2026
@chinmaydd chinmaydd added the comgr Related to Code Object Manager label May 8, 2026
Copy link
Copy Markdown

@chinmaydd chinmaydd left a comment

Choose a reason for hiding this comment

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

LGTM, thanks !

@z1-cciauto
Copy link
Copy Markdown
Collaborator

@chinmaydd
Copy link
Copy Markdown

!PSDB

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

Labels

comgr Related to Code Object Manager

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants