[PATCH] D52258: Fix for bug 34002
Oliver Stannard via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 19 01:36:28 PDT 2018
olista01 added inline comments.
================
Comment at: lib/Target/ARM/AsmParser/ARMAsmParser.cpp:9457
// not legal to branch into an IT block.
flushPendingInstructions(getStreamer());
if (NextSymbolIsThumb) {
----------------
Can this call be removed now? doBeforeLabelEmit and onLabelParsed are only ever called in pairs before and after emitting the label, so there should never be anything to flush here, and as you've pointed out doing it here is incorrect.
================
Comment at: test/MC/ARM/implicit-it-generation.s:70
@ Flush on a label
.section test6
----------------
I think this test already exposes the bug, we just need to add CHECK lines to make sure the labels are emitted in the correct place. That would be clearer than adding a second test to check the label locations.
Repository:
rL LLVM
https://reviews.llvm.org/D52258
More information about the llvm-commits
mailing list