<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 11, 2019 at 6:26 AM Martin Storsjö <<a href="mailto:martin@martin.st">martin@martin.st</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">Hi,<br>
<br>
The COFF support in llvm-objcopy is in a pretty half-finished state at the <br>
moment. I had hoped to have it mostly usable for the common scenarios by <br>
the time of the branch (the initial patch was sent at the end of <br>
November), but it's still lacking stripping of sections (while stripping <br>
of symbols is pretty much done) and a few other minor features I have <br>
waiting to be polished up according to review comments.<br>
<br>
When used right now, it won't error out or warn about not doing what <br>
actually was requested, but just copy the object/executable and do some <br>
symbol removals if requested.<br>
<br>
With the branch coming real soon, what's the preferred course of action?<br>
<br>
0 - Do nothing, release this as is. As llvm-objcopy hasn't supported COFF <br>
before, nobody will try it and run into the issue.<br></blockquote><div>Option 0 seems fine to me</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
1 - Rip out the COFF support in the branch and let it error out as it did <br>
before.<br>
<br>
2 - Add a custom patch for the branch, which makes the tool error out if <br>
an option that is known to be unimplemented/incomplete is specified, <br>
keeping the functionality that is known to be complete.<br>
<br>
3 - Merge the llvm-objcopy/COFF patches to the release branch as things <br>
are completed in trunk during the rc phase, hopefully reaching a more <br>
complete state within that timeframe. As this is a new feature, it <br>
shouldn't have any stability impact on the rest (at least as long as we <br>
don't end up needing to patch other libraries in the process).<br>
<br>
What do you think?<br>
<br>
// Martin<br>
<br>
On Wed, 9 Jan 2019, Hans Wennborg via cfe-dev wrote:<br>
<br>
> Hello everyone,<br>
><br>
> This is just a quick reminder that the upcoming release branch is<br>
> scheduled for creation one week from today, on Wednesday 16 January<br>
> 2019.<br>
><br>
> The full schedule is available under "Upcoming Releases" in the right<br>
> column at <a href="https://llvm.org" rel="noreferrer" target="_blank" class="cremed">https://llvm.org</a>.<br>
><br>
> Please try to avoid committing disruptive changes just before or after<br>
> the branch.<br>
><br>
> As the branch is created, the trunk version will become 9.0.0.<br>
><br>
> Thanks,<br>
> Hans<br>
> _______________________________________________<br>
> cfe-dev mailing list<br>
> <a href="mailto:cfe-dev@lists.llvm.org" target="_blank" class="cremed">cfe-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank" class="cremed">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div></div>