mstorsjo added a comment. In D111707#3062361 <https://reviews.llvm.org/D111707#3062361>, @MaskRay wrote:
> I was thinking of when testing "windows" x "pic", whether the test should > reside in "windows" or "pic". > If in "windows", we can decrease the number of RUN lines and use one RUN line > to test multiple properties at one time. > If in "pic", it does make it clear what platforms default to pic but I think > many platforms have dedicated tests and duplicate the coverage here anyway. True, but then again, this avoids needing to duplicate the PIC level checks across two files. It costs a couple more `RUN` lines, true... Anyway, I'm not strongly opposed to restructuring it, but feel free to pick that up as a followup if you want to - I'd rather keep this one as-is. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111707/new/ https://reviews.llvm.org/D111707 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits