[PATCH] D50839: [llvm] Optimize YAML::isNumeric

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev updated this revision to Diff 161204.
kbobyrev added a comment.

Add couple tests, fix formatting issues, use `__builtin_trap()` instead of `assert` in fuzzer so that it's more transparent.

Also, fuzzing this unreadable version for a couple of hours suggests that it is valid.


https://reviews.llvm.org/D50839

Files:
  llvm/include/llvm/Support/YAMLTraits.h
  llvm/tools/llvm-yaml-numeric-parser-fuzzer/CMakeLists.txt
  llvm/tools/llvm-yaml-numeric-parser-fuzzer/DummyYAMLNumericParserFuzzer.cpp
  llvm/tools/llvm-yaml-numeric-parser-fuzzer/yaml-numeric-parser-fuzzer.cpp
  llvm/unittests/Support/YAMLIOTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50839.161204.patch
Type: text/x-patch
Size: 10598 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180817/022b7b62/attachment.bin>