Skip to content

Conversation

@causten
Copy link
Collaborator

@causten causten commented Oct 8, 2025

Motivation

Modified env file to label a pass was code

Technical Details

From rewrite_pooling to rewrite_pooling

Changelog Category

    • Added: New functionality.
    • Changed: Changes to existing functionality.
    • Removed: Functionality or support that has been removed. (Compared to a previous release)
    • Optimized: Component performance that has been optimized or improved.
    • Resolved Issues: Known issues from a previous version that have been resolved.
    • Not Applicable: This PR is not to be included in the changelog.

@causten causten requested a review from a team as a code owner October 8, 2025 13:36
@causten causten requested a review from Copilot October 8, 2025 13:36
Copy link

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 corrects documentation formatting by properly marking the pass name as code. The change applies proper reStructuredText code formatting to the rewrite_pooling pass name to maintain consistency with documentation standards.

  • Applied proper code formatting to the rewrite_pooling pass name using double backticks

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

* - | ``MIGRAPHX_REWRITE_LRN``
| Turns on LRN-to-pooling lowering in the rewrite_pooling pass.
| Turns on LRN-to-pooling lowering in the ``rewrite_pooling`` pass.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a bit ambiguous. Is it turning on the lowering of LRN-to-pooling? Or is it turning on "LRN-to-pooling lowering"?

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