search

besser82 / libxcrypt

Extended crypt library for descrypt, md5crypt, bcrypt, and others
GNU Lesser General Public License v2.1
189 stars 52 forks source link

doc: Update LTO support status in README.md #176

Closed xry111 closed 6 months ago

zackw commented 6 months ago

With the text as is, it would not be correct to merge this patch until after #109 is merged. The necessary compiler support now exists, but libxcrypt must also be changed to use that support before we can say "LTO is supported".

@xry111 If you would like to revise your change so that it accurately summarizes the present state of affairs (see #24 as well as #109), then I'll consider merging it.

xry111 commented 6 months ago

With the text as is, it would not be correct to merge this patch until after #109 is merged. The necessary compiler support now exists, but libxcrypt must also be changed to use that support before we can say "LTO is supported".

@xry111 If you would like to revise your change so that it accurately summarizes the present state of affairs (see #24 as well as #109), then I'll consider merging it.

We are already using that support since 38f7f06ec358561a01636f0d990ed98051b9120a, AFAIK. But #109 already contains the wanted README.md change so I'll close this.