[PATCH] D79107: Addressing a very strict assert check in CostAnnotationWriter::emitInstructionAnnot

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 29 11:49:04 PDT 2020


mtrofin accepted this revision.
mtrofin added a comment.
This revision is now accepted and ready to land.

lgtm



================
Comment at: llvm/lib/Analysis/InlineCost.cpp:737
+        OS << "; No analysis for the instruction\n";
+        return ;
+    }
----------------
Nit: remove space between return and ; (probably clang-format would handle it)


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

https://reviews.llvm.org/D79107





More information about the llvm-commits mailing list