[patch] Fix pr24486
Maxim Ostapenko via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 24 04:11:53 PDT 2015
Here is another patch to fix pr24486. Could you please take a look when
you have a time?
On 08/09/15 20:18, Rafael Espíndola wrote:
> What I think is needed is a more invasive change from Section to
> Fragment. So for example,
> Would become
> and there wouldn't be a single dummy fragment. The asm streamer would
> create one for each section it creates.
I thought it would be better to replace MCStreamer::AssignSection with
MCStreamer::AssignSectionAndFragment, because for some symbols we still
want to set section explicitly. Does this make sense?
This patch works fine for me and passed make check-all on my
x86_64-unknown-linux-gnu box as well as simple bootstrap (I've rebuilt
clang three times by itself).
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 30704 bytes
Desc: not available
More information about the llvm-commits