[PATCH] D63518: BitStream reader: propagate errors

Alex Brachet via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 27 00:10:38 PDT 2019


abrachet added inline comments.


================
Comment at: llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp:205
+      return MaybeBitCode.takeError();
+    switch (unsigned BitCode = MaybeBitCode.get()) {
     default: // Default behavior: reject
----------------
This and an identical switch on line 5367 cause an unused variable warning from this commit. I don't know if the build bots report on this, or the proper way to tell you about this but hopefully you will see it :)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63518





More information about the cfe-commits mailing list