[PATCH] D128754: [llvm] Remove unused and redundant crc32 funcction from llvm::compression::zlib namespace

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 1 13:34:24 PDT 2022


MaskRay accepted this revision.
MaskRay added a comment.

Thanks!



================
Comment at: llvm/lib/Support/Compression.cpp:25
 using namespace llvm;
 
 using namespace llvm::compression;
----------------
if a previous patch introduces a blank line between two `using`. please drop it from the previous patch.


================
Comment at: llvm/unittests/Support/CompressionTest.cpp:21
 using namespace llvm;
 
 using namespace llvm::compression;
----------------
if a previous patch introduces a blank line between two using. please drop it from the previous patch.


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

https://reviews.llvm.org/D128754



More information about the cfe-commits mailing list