Changelog fixes - #5200
Merged
Merged
Conversation
CharlieL7
requested review from
TedThemistokleous and
causten
and
a lite review from Copilot
August 27, 2026 18:01
CharlieL7
added
skip bot checks
Contributor
There was a problem hiding this comment.
Pull request overview
Fixes and standardizes the Develop section of CHANGELOG.md by correcting/adding PR references (including the dyn_slice PR number) and cleaning up inconsistent changelog entry formatting.
Changes:
- Corrected/added missing PR numbers for multiple changelog bullets (including
dyn_slice). - Standardized bullet formatting (punctuation and consistent entry structure).
- Added a few missing
Developchangelog entries to align the release notes with implemented work.
Suppressed comments (3)
CHANGELOG.md:115
- The activation name is typically written “SiLU” (capitalization) in technical docs; consider adjusting this entry to avoid looking like a typo.
* Fuse expert Silu Heads (MoE) into batched GEMM via fuse_horizontal (#5087).
CHANGELOG.md:63
- Capitalize product names consistently (“Netron”, “Python”) to match other entries in this changelog.
* Updated netron output to create an ONNX-like protobuf. Now also includes debug symbols if enabled (#4701).
* Updated python API to allow getting and adding debug symbols from instructions (#4803).
CHANGELOG.md:112
- These bullets are in past tense everywhere else in this section; “Add matcher” reads like an imperative and should be “Added matcher”.
* Add matcher to simplify_algebra to find and replace pow(x, 2) with mul(x, x) (#4681).
* Add matcher to `fuse_attention` that removes Q/DQ pairs from attention blocks (#4900).
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| * Added find_concat_same_input matcher to convert concat(N*x) into multibroadcast(x) to reduce hipCopy() (#4981) | ||
| * Added debug symbols for MIGraphX instructions such that parsed and compiled instructions can be tracked back to their ONNX origin node (#4626). | ||
| * Added environment variable `MIGRAPHX_GPU_DUMP_BENCHMARK_MXR` to dump mxr files for benchmarking (#4766). | ||
| * Added FusedMatMul microsoft contrib operator (#4806). |
Contributor
There was a problem hiding this comment.
@CharlieL7 This line is from my change. Do you think it would be better to capitalize Microsoft. If so, I can commit this suggestion.
Thanks for doing this effort.
TedThemistokleous
approved these changes
Aug 27, 2026
ahsan-ca
approved these changes
Aug 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
dyn_slicePR.Technical Details
Changelog Category
Add a
CHANGELOG.mdentry for any option other thanNot ApplicableFollow the LLVM AI Tool Use Policy for contributions using AI.