[PATCH] D121512: [Support] Change zlib::compress to return void

2022-03-14 Thread Fangrui Song via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG407c721ceb93: [Support] Change zlib::compress to return void (authored by MaskRay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D121512: [Support] Change zlib::compress to return void

2022-03-14 Thread Igor Kudrin via Phabricator via cfe-commits
ikudrin accepted this revision. ikudrin added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121512/new/ https://reviews.llvm.org/D121512 _

[PATCH] D121512: [Support] Change zlib::compress to return void

2022-03-14 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay updated this revision to Diff 415167. MaskRay marked an inline comment as done. MaskRay added a comment. Simplify CompressedSection::CompressedSection in llvm-objcopy. CompressedSection::create can be further simplified, but I'll leave that as a future change. Repository: rG LLVM Git

[PATCH] D121512: [Support] Change zlib::compress to return void

2022-03-14 Thread Igor Kudrin via Phabricator via cfe-commits
ikudrin added inline comments. Comment at: llvm/lib/ObjCopy/ELF/ELFObject.cpp:568 DebugCompressionType CompressionType, Error &OutErr) : SectionBase(Sec), CompressionType(CompressionType), --

[PATCH] D121512: [Support] Change zlib::compress to return void

2022-03-13 Thread Joerg Sonnenberger via Phabricator via cfe-commits
joerg added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121512/new/ https://reviews.llvm.org/D121512 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[PATCH] D121512: [Support] Change zlib::compress to return void

2022-03-12 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay updated this revision to Diff 414896. MaskRay retitled this revision from "[Support] zlib::compress: replace compress2 with iterative deflate" to "[Support] Change zlib::compress to return void". MaskRay edited the summary of this revision. MaskRay added a comment. Herald added subscriber