[PATCH] D81422: Change filecheck default to dump input on failure
Mehdi AMINI via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 9 20:57:27 PDT 2020
mehdi_amini added a comment.
In D81422#2083882 <https://reviews.llvm.org/D81422#2083882>, @arsenm wrote:
> 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.
This is something that the script generating every combination could manage to split as well?
Alternatively, can these very large test be appended `--dump-input=never` on the RUN line? (maybe the test generator can do this?)
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