<div dir="ltr">> 1. Haowei's proposal is a good first step. A reproducer is good for 100% crashes. The reproducer archive is good for developers that need to iterate when fixing a crash. That can be achieved by mimicking the behavior of -fintegrated-cc1 and CC1Command (which is somehow equivalent to safeLldMain()), followed by something similar to generateCompilationDiagnostics(). All this controlled by an optional cmd-line flag. Also worth noting that some LLD drivers (if not all) support reproducers already.<br><div><br></div><div>I think implementing option 1 as a starter is a good idea. The existing crash reproducer for clang with '-fintegrated-cc1' works pretty good in our case so if we can have it in lld, It should greatly improve the debug experience. If there is no further objection, I can start working on this.</div><div><br></div><div>> Sounds like a good idea. I suppose in case of a crash, Driver::generateCompilationDiagnostics() could invoke the linker again with --reproduce? That would be the "cheap" version of option 2.</div><div><br></div><div>I agree with this as well. Though we probably won't need a perfect crash reproducer, this solution is simple enough and it would cover some rare cases that an in process lld crash handler cannot handle.</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 21, 2021 at 10:56 AM Alexandre Ganea <<a href="mailto:alexandre.ganea@ubisoft.com">alexandre.ganea@ubisoft.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The build system needs to be aware of the core dump files, and deal with them before/after the build. It should do an automatic cleanup before and compress+upload the core dumps to a network location if a crash occurs. I agree that the size is a problem if that isn't dealt with. 50 GB .dmp files for LLD crashes aren't uncommon, and you would probably see the same figures with production builds on Linux. Mostly likely dumps would need to be enabled explicitly with an env.var or a system setting on the build system? I personally find it valuable to have .dmp files. As hard as we try to make the CI builds the same as the local ones, there's always something different that triggers a different behavior on the build system.<br>
<br>
-----Message d'origine-----<br>
De : Fangrui Song <<a href="mailto:maskray@google.com" target="_blank">maskray@google.com</a>> <br>
Envoyé : April 21, 2021 12:56 PM<br>
À : Alexandre Ganea <<a href="mailto:alexandre.ganea@ubisoft.com" target="_blank">alexandre.ganea@ubisoft.com</a>><br>
Cc : David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>>; Petr Hosek <<a href="mailto:phosek@chromium.org" target="_blank">phosek@chromium.org</a>>; Haowei Wu <<a href="mailto:haowei@google.com" target="_blank">haowei@google.com</a>>; <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
Objet : Re: [llvm-dev] [RFC] Generating LLD reproducers on crashes<br>
<br>
For 3: on Linux, there are many system effects which can suppress crash dumps...<br>
I am still abit concerned with the size of the core file..<br>
If users find this particular useful and want to go down this route, it is fine to me.<br>
<br>
</blockquote></div>