[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