[PATCH] D123100: [Support/Hash functions] Change the `final()` and `result()` of the hashing functions to return an array of bytes

[Argyrios Kyrtzidis via Phabricator via llvm-commits]

akyrtzi updated this revision to Diff 420359.
akyrtzi added a comment.

Adjust `ASTFileSignature` to accept only the array hash bytes and directly from the `final()` call.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123100

Files:
  bolt/lib/Core/DebugData.cpp
  clang/include/clang/Basic/Module.h
  clang/lib/Serialization/ASTWriter.cpp
  lld/ELF/Writer.cpp
  lld/MachO/SyntheticSections.cpp
  llvm/include/llvm/Support/BLAKE3.h
  llvm/include/llvm/Support/HashBuilder.h
  llvm/include/llvm/Support/MD5.h
  llvm/include/llvm/Support/SHA1.h
  llvm/include/llvm/Support/SHA256.h
  llvm/include/llvm/Support/raw_sha1_ostream.h
  llvm/lib/Bitcode/Reader/BitcodeAnalyzer.cpp
  llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
  llvm/lib/DebugInfo/CodeView/TypeHashing.cpp
  llvm/lib/ObjCopy/MachO/MachOWriter.cpp
  llvm/lib/Support/BLAKE3/README.md
  llvm/lib/Support/MD5.cpp
  llvm/lib/Support/SHA1.cpp
  llvm/lib/Support/SHA256.cpp
  llvm/unittests/Support/BLAKE3Test.cpp
  llvm/unittests/Support/HashBuilderTest.cpp
  llvm/unittests/Support/MD5Test.cpp
  llvm/unittests/Support/SHA256.cpp
  llvm/unittests/Support/raw_sha1_ostream_test.cpp
  mlir/lib/Pass/IRPrinting.cpp

STAMPS
actor(@akyrtzi) application(Differential) author(@akyrtzi) herald(H105) herald(H123) herald(H208) herald(H224) herald(H225) herald(H311) herald(H423) herald(H452) herald(H530) herald(H532) herald(H533) herald(H538) herald(H540) herald(H541) herald(H544) herald(H546) herald(H547) herald(H550) herald(H551) herald(H554) herald(H557) herald(H560) herald(H561) herald(H565) herald(H576) herald(H592) herald(H597) herald(H598) herald(H607) herald(H610) herald(H615) herald(H625) herald(H629) herald(H645) herald(H672) herald(H685) herald(H688) herald(H698) herald(H723) herald(H727) herald(H729) herald(H739) herald(H802) herald(H805) herald(H809) herald(H827) herald(H842) herald(H844) herald(H845) herald(H847) herald(H854) herald(H855) herald(H861) herald(H864) monogram(D123100) object-type(DREV) phid(PHID-DREV-ngvkfimyomxukhcmhald) reviewer(#lld-macho) reviewer(@Amir) reviewer(@dexonsmith) reviewer(@jhenderson) reviewer(@maksfb) reviewer(@MaskRay) reviewer(@rafauler) reviewer(@rriddle) revision-repository(rG) revision-status(accepted) subscriber(@aartbik) subscriber(@antiagainst) subscriber(@arichardson) subscriber(@arpith-jacob) subscriber(@ayermolo) subscriber(@cfe-commits) subscriber(@Chia-hungDuan) subscriber(@cota) subscriber(@dcaballe) subscriber(@dexonsmith) subscriber(@emaste) subscriber(@grosul1) subscriber(@hiraditya) subscriber(@Joonsoo) subscriber(@jurahul) subscriber(@Kayjukh) subscriber(@liufengdb) subscriber(@llvm-commits) subscriber(@mehdi_amini) subscriber(@mgester) subscriber(@msifontes) subscriber(@nicolasvasilache) subscriber(@rdzhabarov) subscriber(@rriddle) subscriber(@sdasgup3) subscriber(@shauheen) subscriber(@StephenFan) subscriber(@stephenneuendorffer) subscriber(@tatianashp) subscriber(@teijeong) subscriber(@wenzhicui) subscriber(@wrengr) subscriber(@yota9) tag(#all) tag(#clang) tag(#lld-macho) tag(#llvm) tag(#mlir) via(conduit)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123100.420359.patch
Type: text/x-patch
Size: 23830 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220405/6e2a13d0/attachment.bin>