[PATCH] D52258: Fix for bug 34002

[Oliver Stannard via Phabricator via llvm-commits]

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