nikic added a comment. In D72538#4296119 <https://reviews.llvm.org/D72538#4296119>, @tejohnson wrote:
> In D72538#4291552 <https://reviews.llvm.org/D72538#4291552>, @nikic wrote: > >> Would it be possible to cut down the Clang side tests to only check parts >> that Clang controls in some way, e.g. that SLPVectorizer is enabled? This >> test is something of a PITA because it tests LLVM implementation details but >> is not part of the LLVM tests, so it usually gets missed when doing pipeline >> changes. > > I think we can cut this down significantly, since the llvm tests added here > check the full ThinLTO default pipeline setup at different opt levels. > However, the point of the clang test is to make sure that for a distributed > ThinLTO backend we correctly invoke thinBackend() which should set up the > ThinLTO default pipeline. So how about I cut this down to check a single pass > that is only invoked in the LTO backends? Specifically, I'm thinking of > WholeProgramDevirtPass. That sounds reasonable to me! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D72538/new/ https://reviews.llvm.org/D72538 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
