[PATCH] D112297: [LTO] Fix assertion failed when flushing bitcode incrementally for LTO output.

Wed Oct 27 21:53:07 PDT 2021

Enna1 added a comment.

In D112297#3092443 <https://reviews.llvm.org/D112297#3092443>, @stephan.yichao.zhao wrote:

> In D112297#3092429 <https://reviews.llvm.org/D112297#3092429>, @MaskRay wrote:
>
>> Agreed. Does this change address this patch's issue (`ld.lld --plugin-opt=emit-llvm -mllvm -bitcode-flush-threshold=0 -o /dev/null %t.o`)?
>>
>> Can you post a patch? (Patching lld/ELF/LTO.cpp gives me a feeling working around a pitfall which might well hit others, so fixing the source would be nice.)
>
> It needs some test to confirm. I probably do not have time to try this until this weekend.
> @Enna1: If you have time, can you please verify if the suggested change works with your local test cases, and update this change if it works?

Sure, I'll verify if the suggested change can fix the assertion failure with my local testcases.
if it works , i'll update this change.
Thanks @stephan.yichao.zhao @MaskRay

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112297/new/

https://reviews.llvm.org/D112297