[PATCH] D58375: [Clang][NewPM] Disable tests that are broken under new PM
Chandler Carruth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 19 17:47:35 PST 2019
chandlerc added a comment.
In D58375#1403144 <https://reviews.llvm.org/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 are failing?
>
> For the tests that do use -O, instead of marking them unsupported, could you use -fno-experimental-new-pass-manager or something like that?
For at least some of them, maybe we should run it in both modes (using the explicit flag as suggested by Eli) and check both forms.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58375/new/
https://reviews.llvm.org/D58375
More information about the cfe-commits
mailing list