[PATCH] D59816: [Support] Implement zlib independent crc32 computation

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 26 05:30:46 PDT 2019


grimar added inline comments.


================
Comment at: lib/DebugInfo/Symbolize/Symbolize.cpp:167
     return false;
-  return !zlib::isAvailable() || CRCHash == zlib::crc32(MB.get()->getBuffer());
+  return CRCHash == (zlib::isAvailable()
+                         ? zlib::crc32(MB.get()->getBuffer())
----------------
What about always using non-zlib implementation?


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

https://reviews.llvm.org/D59816





More information about the llvm-commits mailing list