diff options
author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-08-21 00:08:10 +0000 |
---|---|---|
committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-08-21 00:08:10 +0000 |
commit | 8f44c98d046a22348504ff11e048753edbeea086 (patch) | |
tree | f67a664ab69d51eea6d1aa99b0361ead12347deb /llvm/lib/Object/MachOObjectFile.cpp | |
parent | 3b40aa57f619771d7e4f5b5267a57c7f8a416bea (diff) | |
download | llvm-8f44c98d046a22348504ff11e048753edbeea086.zip llvm-8f44c98d046a22348504ff11e048753edbeea086.tar.gz llvm-8f44c98d046a22348504ff11e048753edbeea086.tar.bz2 |
ARM: Avoid dereferencing end() in ARMFrameLowering::emitEpilogue
This fixes the crash from PR29072, where the MachineBasicBlock::iterator
wasn't being properly checked against MachineBasicBlock::end() before
iterating. This was another bug exposed by the new
ilist::iterator::operator*() assertion from r279314.
This testcase is poor quality. bugpoint couldn't reduce any further,
and I haven't had time to dig into what's going on so I can't invent a
better one. I didn't even get good CHECK lines in: this is just a
crasher.
I'm committing anyway since this is a real crash with an obvious fix,
but I'll leave PR29072 open and ask an ARM maintainer to help improve
the testcase.
llvm-svn: 279391
Diffstat (limited to 'llvm/lib/Object/MachOObjectFile.cpp')
0 files changed, 0 insertions, 0 deletions