[PATCH] D58374: [Clang][NewPM] Don't bail out if the target machine is empty

2019-05-06 Thread Petr Hosek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC360100: [Clang][NewPM] Don't bail out if the target machine is empty (authored by phosek, committed by ). Changed prior to commit: https://reviews.llvm.org/D58374?vs=188999&id=198367#toc Repository:

[PATCH] D58374: [Clang][NewPM] Don't bail out if the target machine is empty

2019-05-06 Thread Chandler Carruth via Phabricator via cfe-commits
chandlerc accepted this revision. chandlerc added a comment. This revision is now accepted and ready to land. LGTM! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58374/new/ https://reviews.llvm.org/D58374 ___ cfe-com

[PATCH] D58374: [Clang][NewPM] Don't bail out if the target machine is empty

2019-03-01 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In D58374#1411376 , @chandlerc wrote: > Maybe update at least some of the tests using these targets to additionally > run with the new pass manager explicitly enabled via flag? Done Repository: rC Clang CHANGES SINCE LAST AC

[PATCH] D58374: [Clang][NewPM] Don't bail out if the target machine is empty

2019-03-01 Thread Petr Hosek via Phabricator via cfe-commits
phosek updated this revision to Diff 188999. phosek marked an inline comment as done. Herald added a subscriber: dschuff. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58374/new/ https://reviews.llvm.org/D58374 Files: clang/lib/CodeGen/BackendUtil.cpp clang/te

[PATCH] D58374: [Clang][NewPM] Don't bail out if the target machine is empty

2019-02-26 Thread Chandler Carruth via Phabricator via cfe-commits
chandlerc requested changes to this revision. chandlerc added a comment. This revision now requires changes to proceed. Maybe update at least some of the tests using these targets to additionally run with the new pass manager explicitly enabled via flag? Comment at: clang/lib/

[PATCH] D58374: [Clang][NewPM] Don't bail out if the target machine is empty

2019-02-26 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. Ping? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D58374/new/ https://reviews.llvm.org/D58374 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/li

[PATCH] D58374: [Clang][NewPM] Don't bail out if the target machine is empty

2019-02-18 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. phosek added a reviewer: chandlerc. Herald added a project: clang. Herald added a subscriber: cfe-commits. This matches the behavior of the old pass manager. There are some targets that don't have target machine at all (e.g. le32, spir) which whose tests would never r