Re: [PATCH] D21954: [PM] Add some internal options for testing out the new PM.

2016-07-18 Thread Sean Silva via cfe-commits
silvas updated this revision to Diff 64420. silvas added a comment. Rebase after r275507. https://reviews.llvm.org/D21954 Files: include/clang/Driver/CC1Options.td include/clang/Frontend/CodeGenOptions.h lib/CodeGen/BackendUtil.cpp lib/CodeGen/CMakeLists.txt lib/Frontend/CompilerInvoc

Re: [PATCH] D21954: [PM] Add some internal options for testing out the new PM.

2016-07-14 Thread Chandler Carruth via cfe-commits
chandlerc added a comment. I'd like someone more active on the Clang side to comment about this, but here is my 2 cents... It seems like it'd be nice to not populate or run or otherwise build up the legacy pass managers when using this option. I guess I would expect something to more split ear

Re: [PATCH] D21954: [PM] Add some internal options for testing out the new PM.

2016-07-14 Thread Sean Silva via cfe-commits
silvas added a comment. Ping. https://reviews.llvm.org/D21954 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D21954: [PM] Add some internal options for testing out the new PM.

2016-07-08 Thread Sean Silva via cfe-commits
silvas added a subscriber: cfe-commits. silvas added a comment. Actually add cfe-commits. Oops. Thanks to Chandler for pointing out I had added llvm-commits instead. http://reviews.llvm.org/D21954 ___ cfe-commits mailing list cfe-commits@lists.llvm