[PATCH] D54780: [llvm-demangle-fuzzer] Also fuzz microsoftDemangle().

Erik Pilkington via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 4 14:03:21 PST 2018


erik.pilkington added a comment.

Hi, thanks for working on this! Fuzzing in the itanium demangler has caught *a lot* of bugs, so this seems pretty useful. Whats the argument for testing the two demanglers in the same path? There isn't really any shared code, so I think it makes more sense to separate these out.

@kcc: is there any oss-fuzz testing for llvm-demangle-fuzzer.cpp? I thought it was just for libcxxabi. Also, will reinterpreting the first char in Data break the existing itanium corpuses?


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

https://reviews.llvm.org/D54780





More information about the llvm-commits mailing list