diff options
author | Ulrich Weigand <ulrich.weigand@de.ibm.com> | 2015-12-16 18:04:06 +0000 |
---|---|---|
committer | Ulrich Weigand <ulrich.weigand@de.ibm.com> | 2015-12-16 18:04:06 +0000 |
commit | 47f3649374aeec6c029c45cd3ab44b47d7e7ea08 (patch) | |
tree | 1b71d22802ac2dede961238a648f6f45fa186028 /llvm/tools/llvm-readobj/llvm-readobj.cpp | |
parent | b51460cf86e730c502a6d6116862a72ed25f5b78 (diff) | |
download | llvm-47f3649374aeec6c029c45cd3ab44b47d7e7ea08.zip llvm-47f3649374aeec6c029c45cd3ab44b47d7e7ea08.tar.gz llvm-47f3649374aeec6c029c45cd3ab44b47d7e7ea08.tar.bz2 |
[SystemZ] Fix assertion failure in adjustSubwordCmp
When comparing a zero-extended value against a constant small enough to
be in range of the inner type, it doesn't matter whether a signed or
unsigned compare operation (for the outer type) is being used. This is
why the code in adjustSubwordCmp had this assertion:
assert(C.ICmpType == SystemZICMP::Any &&
"Signedness shouldn't matter here.");
assuming the the caller had already detected that fact. However, it
turns out that there cases, in particular with always-true or always-
false conditions that have not been eliminated when compiling at -O0,
where this is not true.
Instead of failing an assertion if C.ICmpType is not SystemZICMP::Any
here, we can simply *set* it safely to SystemZICMP::Any, however.
llvm-svn: 255786
Diffstat (limited to 'llvm/tools/llvm-readobj/llvm-readobj.cpp')
0 files changed, 0 insertions, 0 deletions