[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