diff options
author | Marius Hillenbrand <mhillen@linux.ibm.com> | 2022-07-25 08:37:31 +0530 |
---|---|---|
committer | Uday Bondhugula <uday@polymagelabs.com> | 2022-07-25 08:37:51 +0530 |
commit | 1ef32e78284bc758112632e9e190b6683ea5b95b (patch) | |
tree | 0318c58968942f43d2aa4ec8c958a38ffcf73e8b /llvm/lib/CodeGen/ProcessImplicitDefs.cpp | |
parent | 73c84f9c13e96a4722ad489e79f8608925cc9e78 (diff) | |
download | llvm-1ef32e78284bc758112632e9e190b6683ea5b95b.zip llvm-1ef32e78284bc758112632e9e190b6683ea5b95b.tar.gz llvm-1ef32e78284bc758112632e9e190b6683ea5b95b.tar.bz2 |
[mlir][Arithmetic] Fix printing larger integer attributes in arith.const
For arith.constant operations of integer type, the operation generates
result names that include the value of the constant (i.e., the
IntegerAttr that defines the constant's value). That code currently
assumes integer widths of 64 bits or less and hits an assert with wider
constants or would create truncated and potentially ambiguous names when
built with assertions disabled.
To enable printing arith.constant ops for arbitrarily wide integer
types, change to use the IntegerAttr's function getValue() when
generating result names.
Also, add a regression test.
Reviewed By: bondhugula
Differential Revision: https://reviews.llvm.org/D129930
Diffstat (limited to 'llvm/lib/CodeGen/ProcessImplicitDefs.cpp')
0 files changed, 0 insertions, 0 deletions