Skip to content

Conversation

dorde-antic
Copy link
Contributor

Motivation

Resolves https://github.com/ROCm/rocMLIR-internal/issues/1984

Technical Details

Remove gfx1101 from nightly and weekly.

Test Plan

Nightly and weekly.

Test Result

Submission Checklist

@dorde-antic dorde-antic marked this pull request as ready for review October 9, 2025 08:47
@dorde-antic dorde-antic requested a review from causten as a code owner October 9, 2025 08:47
@dorde-antic dorde-antic changed the title Remove gfx1101 when nightly or weekly Don't pick gfx1101 on nightly and weekly runs Oct 9, 2025
label = 'mlir && ( gfx1100 || gfx1101 )'
}
} else if (codepath == "navi4x") {
label = 'mlir && ( gfx1200 || gfx1201 )'
Copy link
Contributor

Choose a reason for hiding this comment

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

should we do the same for gfx12? and navi2?

case "gfx1030":
return shouldRunFromCodepath("navi21")
case "gfx1100":
case "gfx1101":
Copy link
Contributor

Choose a reason for hiding this comment

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

does this affect PR CI as well?

@umangyadav umangyadav requested a review from Copilot October 10, 2025 13:17
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes support for the gfx1101 graphics chip from nightly and weekly Jenkins CI runs to resolve an internal issue. The change restricts automated builds to use only gfx1100 for navi3x codepath testing during scheduled runs while maintaining support for both chips in regular builds.

  • Modified navi3x label selection to exclude gfx1101 for nightly/weekly runs
  • Removed gfx1101 from the chip-to-codepath mapping function

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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