Skip to content

[Infra][Nightly] Temp cherry-pick 2719 for hipcc and hiprtc test#2727

Open
kirthana14m wants to merge 1 commit into
amd-stagingfrom
amd/dev/kirthana14m/nightly-temp-devicelib-fix
Open

[Infra][Nightly] Temp cherry-pick 2719 for hipcc and hiprtc test#2727
kirthana14m wants to merge 1 commit into
amd-stagingfrom
amd/dev/kirthana14m/nightly-temp-devicelib-fix

Conversation

@kirthana14m
Copy link
Copy Markdown
Collaborator

No description provided.

run: |
cd compiler/amd-llvm
git fetch origin pull/2719/head:pr-2719
git cherry-pick pr-2719
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Wondering wouldn't be better to add error handling (in the future the PR and the tip can diverge). Something like

    if git cherry-pick pr-2719 --no-edit; then
      cd ../..
      git add compiler/amd-llvm
      git commit -m "Update amd-llvm submodule with PR 2719"
    else
      echo "::warning::Cherry-pick of PR 2719 failed (likely conflicts). Skipping temp fix."
      git cherry-pick --abort
    fi

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

On the other hand I think stopping the build is fine

run: |
cd compiler/amd-llvm
git fetch origin pull/2719/head:pr-2719
git cherry-pick pr-2719
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

On the other hand I think stopping the build is fine

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