[PATCH] D27411: [Assembler] Better error messages for .org directive

[Oliver Stannard via Phabricator via llvm-commits]

olista01 added inline comments.


================
Comment at: lib/MC/MCAssembler.cpp:925
+    Layout.getFragmentOffset(&*Section.rbegin());
+    computeFragmentSize(Layout, *Section.rbegin());
   }
----------------
rengolin wrote:
> why do you need to compute the fragment size? Is it for the FragmentOffset parameter in the error message?
The errors are emitted while calculating the fragment size. Previously this just called getFragmentOffset for the last fragment in each section, which caused the size of every fragment except the last to be calculated. I found some cases where the size of the last fragment wasn't evaluated until a later stage. This forces the errors to all be detected at a consistent time.


Repository:
  rL LLVM

https://reviews.llvm.org/D27411