- add logically missing test cases;
- add appropriate comments;
- add appropriate TODO's.
See initial motivation in https://reviews.llvm.org/D117302
Paths
| Differential D118769
Split fast-basictest.ll according to passes responsible for optimizations ClosedPublic Authored by kovdan01 on Feb 2 2022, 3:13 AM.
Details Summary
See initial motivation in https://reviews.llvm.org/D117302
Diff Detail
Event TimelineComment Actions I didn't go over each test, but this seems good to me in providing better visibility about what each pass is responsible for. For the files that still run multiple passes, I think we're trying to show that the combination of passes are cooperating to produce the expected/optimal result. Comment Actions
Thanks for this remark, moving such tests to llvm/test/Transforms/PhaseOrdering makes sense. Regarding using -O2 instead of naming specific passes - I suppose that both options have their benefits (we want to know if something breaks both in the whole pipeline and in a specific combination of passes), so I think that we can test both variants with --check-prefix opt's option. Such approach is already taken, for example, in X86/pr52078.ll. Please let me know if you approve such approach, and if yes - I'll do that. Comment Actions
Yes, we can use >1 RUN line for more coverage. There's little extra cost with that approach. Comment Actions @spatel Fixed the issue we were talking about - could you give a review on the updated patch please?
This revision is now accepted and ready to land.Feb 3 2022, 7:00 AM This revision was landed with ongoing or failed builds.Feb 4 2022, 1:22 AM Closed by commit rG8471c537d55d: Split fast-basictest.ll according to passes responsible for optimizations (authored by kovdan01, committed by asavonic). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 405890 llvm/test/Transforms/InstCombine/fast-basictest.ll
llvm/test/Transforms/PhaseOrdering/fast-basictest.ll
llvm/test/Transforms/PhaseOrdering/fast-reassociate-gvn.ll
llvm/test/Transforms/Reassociate/fast-basictest.ll
|
We can reduce the number (maybe eliminate all) of the duplicated check lines by using a shared prefix. Try something like this: