Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge with fixes to be006372 (30) #289

Merged
merged 2 commits into from
Aug 21, 2024
Merged

Conversation

cferry-AMD
Copy link
Collaborator

No description provided.

andrey-golubev and others added 2 commits April 9, 2024 02:08
…lvm#85766)

At present, large ElementsAttr is unconditionally printed with a hex
string. This means that in IR large constant values often look like:
dense<"0x000000000004000000080000000004000000080000000..."> :
tensor<10x10xi32>

Hoisting hex printing control to the user level for tooling means that
one can disable the feature and get human-readable values when
necessary:
dense<[16, 32, 48, 500...]> : tensor<10x10xi32>

Note: AsmPrinterOptions::printElementsAttrWithHexIfLarger is not always
possible to be used as it requires that one exposes MLIR's command-line
options in user tooling (including an actual compiler).

Co-authored-by: Harald Rotuna <harald.razvan.rotuna@intel.com>
Base automatically changed from bump_to_3b74f8c1 to feature/fused-ops August 21, 2024 20:24
An error occurred while trying to automatically change base from bump_to_3b74f8c1 to feature/fused-ops August 21, 2024 20:24
@mgehre-amd mgehre-amd merged commit 4fa5aa7 into feature/fused-ops Aug 21, 2024
4 checks passed
@mgehre-amd mgehre-amd deleted the bump_to_be006372 branch August 21, 2024 20:25
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