[PATCH] D96164: [lld-macho] add code signature for native arm64 macOS

Greg McGary via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 7 22:01:48 PST 2021


gkm added a comment.

In D96164#2547480 <https://reviews.llvm.org/D96164#2547480>, @smeenai wrote:

> In D96164#2547390 <https://reviews.llvm.org/D96164#2547390>, @thakis wrote:
>
>> LLVM tries to not depend on external libraries if possible. Given that sha256 isn't that much code (https://golang.org/src/crypto/sha256/), just adding a fresh implementation of it (to lld/MachO for now) is probably the way to go.
>
> Agreed. There's some prior art here too ... LLVM has a SHA1 implementation in `llvm/include/llvm/Support/SHA1.h` and `llvm/lib/Support/SHA1.cpp`. That was adapted from an existing public-domain implementation, and we could do the same here (find a SHA256 implementation with a license that allows it to copied into LLVM).

Openssl has a sha256 implementation with Apache 2.0 license. Unless someone knows of a better candidate, I'll go with that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96164



More information about the llvm-commits mailing list