[PATCH] D81422: Change filecheck default to dump input on failure

Matt Arsenault via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 9 18:48:25 PDT 2020


arsenm added a comment.

In D81422#2083808 <https://reviews.llvm.org/D81422#2083808>, @mehdi_amini wrote:

> In D81422#2083761 <https://reviews.llvm.org/D81422#2083761>, @arsenm wrote:
>
> > I think this is a worse default for development for large tests.
>
>
> Maybe the issue is with large tests that needs to be broken up?


This isn't really manageable, especially with the trend of using update_* test checks scripts. Stuff like legalization tests just have to stress every combination of inputs. The -dump-input-on-failure could also be smarter about how much context it prints

> To me this is a general improvement during development at my desk and reading a test output to understand the failure: this goes beyond build bots.
> 
> (you have the environment variable you can set locally if you don't want the behavior though)

The environment variable was removed though? I would also at least expect this to be an option I can set at cmake time and never have to think about again


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81422





More information about the cfe-commits mailing list