[PATCH] D38036: [yaml2obj] - Don't crash on invalid document.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 21 13:39:04 PDT 2017


ruiu added inline comments.


================
Comment at: lib/ObjectYAML/ObjectYAML.cpp:38
+    if (!((Input &)IO).getCurrentNode())
+      return;
+
----------------
I think this is not correct. You need to set an error because this is an error condition.


https://reviews.llvm.org/D38036





More information about the llvm-commits mailing list