[PATCH] D91372: Some updates/fixes to the creduce script.

2020-11-12 Thread Amy Huang via Phabricator via cfe-commits
akhuang updated this revision to Diff 304964. akhuang marked an inline comment as done. akhuang added a comment. address comment Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91372/new/ https://reviews.llvm.org/D91372 Files:

[PATCH] D91372: Some updates/fixes to the creduce script.

2020-11-12 Thread Amy Huang via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0c80b542d38b: Some updates/fixes to the creduce script. (authored by akhuang). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D91372: Some updates/fixes to the creduce script.

2020-11-12 Thread George Burgess IV via Phabricator via cfe-commits
george.burgess.iv accepted this revision. george.burgess.iv added a comment. This revision is now accepted and ready to land. thanks! Comment at: clang/utils/creduce-clang-crash.py:156 + def skip_function(func_name): +for name in filters: + if name in

[PATCH] D91372: Some updates/fixes to the creduce script.

2020-11-12 Thread Amy Huang via Phabricator via cfe-commits
akhuang created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. akhuang requested review of this revision. This was motivated by changes to llvm's `not --crash` disabling symbolization but I ended up removing `not` from the script entirely because it returns