[PATCH] D58375: [Clang][NewPM] Disable tests that are broken under new PM

efriedma added a comment. I thought the plan was to fix whatever issue was causing the -O0 tests to fail, then rebase on top of that? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58375/new/ https://reviews.llvm.org/D58375

[PATCH] D58375: [Clang][NewPM] Disable tests that are broken under new PM

phosek added a comment. Ping? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58375/new/ https://reviews.llvm.org/D58375 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/li

[PATCH] D58375: [Clang][NewPM] Disable tests that are broken under new PM

phosek added a comment. In D58375#1403144 , @efriedma wrote: > I can understand why tests that use -O1 or -O2 would produce different > results with the new pass manager, but it looks like not all the tests are > like that. Do you know why those tests a

[PATCH] D58375: [Clang][NewPM] Disable tests that are broken under new PM

phosek updated this revision to Diff 187493. Herald added a subscriber: mstorsjo. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58375/new/ https://reviews.llvm.org/D58375 Files: clang/test/CMakeLists.txt clang/test/CodeGen/aarch64-neon-across.c clang/test/Co

[PATCH] D58375: [Clang][NewPM] Disable tests that are broken under new PM

chandlerc added a comment. In D58375#1403144 , @efriedma wrote: > I can understand why tests that use -O1 or -O2 would produce different > results with the new pass manager, but it looks like not all the tests are > like that. Do you know why those test

[PATCH] D58375: [Clang][NewPM] Disable tests that are broken under new PM

chandlerc requested changes to this revision. chandlerc added a comment. This revision now requires changes to proceed. Based on the -dev discussion, update once the target machine differences are addressed by mimicing the way the legacy PM works, which will hopefully restrict this similarly to

[PATCH] D58375: [Clang][NewPM] Disable tests that are broken under new PM

efriedma added a comment. I can understand why tests that use -O1 or -O2 would produce different results with the new pass manager, but it looks like not all the tests are like that. Do you know why those tests are failing? For the tests that do use -O, instead of marking them unsupported, cou

[PATCH] D58375: [Clang][NewPM] Disable tests that are broken under new PM

phosek created this revision. phosek added a reviewer: chandlerc. Herald added subscribers: cfe-commits, jdoerfert, kbarton, eraman, javed.absar, mgorny, nemanjai. Herald added a reviewer: serge-sans-paille. Herald added a project: clang. Not every pass has been ported to new PM which is breaking