aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/CodeGen/MachineVerifier.cpp
diff options
context:
space:
mode:
authorStanislav Mekhanoshin <Stanislav.Mekhanoshin@amd.com>2019-11-04 12:41:31 -0800
committerStanislav Mekhanoshin <Stanislav.Mekhanoshin@amd.com>2019-11-05 10:25:27 -0800
commitde56a890725713dffc4ab5bf5fb2f434df27ed4d (patch)
tree5317ffd53b22b58ee72801642beaa0f7e016c6f0 /llvm/lib/CodeGen/MachineVerifier.cpp
parent87e0cb4f1ad299c87c3e26676a9b31b3caf58921 (diff)
downloadllvm-de56a890725713dffc4ab5bf5fb2f434df27ed4d.zip
llvm-de56a890725713dffc4ab5bf5fb2f434df27ed4d.tar.gz
llvm-de56a890725713dffc4ab5bf5fb2f434df27ed4d.tar.bz2
[AMDGPU] return Fail instead of SolfFail from addOperand()
addOperand() method of AMDGPU disassembler returns SoftFail on error. All instances which may lead to that place are an impossible encdoing, not something which is possible to encode, but semantically incorrect as described for SoftFail. Then tablegen generates a check of the following form: if (Decode...(..) == MCDisassembler::Fail) { return MCDisassembler::Fail; } Since we can only return Success and SoftFail that is dead code as detected by the static code analyzer. Solution: return Fail as it should be. See https://bugs.llvm.org/show_bug.cgi?id=43886 Differential Revision: https://reviews.llvm.org/D69819
Diffstat (limited to 'llvm/lib/CodeGen/MachineVerifier.cpp')
0 files changed, 0 insertions, 0 deletions