[PATCH] D61608: Fix YAML parser's Document::skip for null nodes

Michael Spencer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 29 15:39:04 PDT 2019


Bigcheese requested changes to this revision.
Bigcheese added a comment.
This revision now requires changes to proceed.

Thanks for finding this!

I don't think this is the correct fix though, as all of the provided inputs should emit an error.  I believe the issue is that `parseBlockNode` is missing a case for unexpected tokens.  Let me know if you need any more info or if you want me to take a look at fixing it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61608





More information about the llvm-commits mailing list