[PATCH] D66832: [Bugpoint][CrashDebugger] Add --same-error to skip non-relevant CodeGenCrash

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a subscriber: mehdi_amini.
vsk added a comment.

In D66832#1664974 <https://reviews.llvm.org/D66832#1664974>, @jsji wrote:

> I think it would be great to make the powerful bugpoint tool easier to use, especially for beginners, for common simple scenarios.
>
> But if we don't think this is really necessary or in the wrong direction of design, I am OK to abandon this patch.


I see the need to make error validation with bugpoint more beginner-friendly, but am worried about situations in which the mode proposed here could confuse beginners (what if the first error isn't the interesting one? or reductions stop too early?). I don't have a better alternative to suggest. Perhaps we could repost @mehdi_amini's writeup in the "Search for a string in the output" section within http://blog.llvm.org/2015/11/reduce-your-testcases-with-bugpoint-and.html, or link to it from the bugpoint docs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66832