diff options
| author | Craig Topper <craig.topper@intel.com> | 2018-12-11 15:32:14 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2018-12-11 15:32:14 +0000 |
| commit | b51283bfd7636c4dd1a894d59f38122571661c64 (patch) | |
| tree | 932e03c31b101b6d984062bdd31c112c37737796 /llvm/tools/llvm-readobj/llvm-readobj.cpp | |
| parent | c79063243751f9b99e1ad51074ddf6eda7ce5ff7 (diff) | |
| download | llvm-b51283bfd7636c4dd1a894d59f38122571661c64.zip llvm-b51283bfd7636c4dd1a894d59f38122571661c64.tar.gz llvm-b51283bfd7636c4dd1a894d59f38122571661c64.tar.bz2 | |
Fix not correct imm operand assertion for SUB32ri in X86CondBrFolding::analyzeCompare
Summary:
When doing X86CondBrFolding::analyzeCompare, it will meet the SUB32ri instruction as below to use the global address for its operand,
%733:gr32 = SUB32ri %62:gr32(tied-def 0), @img2buf_normal, implicit-def $eflags
JNE_1 %bb.41, implicit $eflags
so the assertion "assert(MI.getOperand(ValueIndex).isImm() && "Expecting Imm operand")" is not correct and change the assert to if make X86CondBrFolding::analyzeCompare return false as not finding the compare for this
Patch by Jianping Chen
Reviewers: smaslov, LuoYuanke, liutianle, Jianping
Reviewed By: Jianping
Subscribers: lebedev.ri, llvm-commits
Differential Revision: https://reviews.llvm.org/D54250
llvm-svn: 348853
Diffstat (limited to 'llvm/tools/llvm-readobj/llvm-readobj.cpp')
0 files changed, 0 insertions, 0 deletions