[PATCH] D28269: [clang] Update lit config in utils/perf-training

2017-01-03 Thread Alexander Shaposhnikov via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL290936: [clang] Update lit config in utils/perf-training (authored by alexshap). Changed prior to commit: https://reviews.llvm.org/D28269?vs=82986&id=82999#toc Repository: rL LLVM https://reviews.ll

[PATCH] D28269: [clang] Update lit config in utils/perf-training

2017-01-03 Thread Chris Bieneman via Phabricator via cfe-commits
beanz added a comment. LGTM! Repository: rL LLVM https://reviews.llvm.org/D28269 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D28269: [clang] Update lit config in utils/perf-training

2017-01-03 Thread Alexander Shaposhnikov via Phabricator via cfe-commits
alexshap updated this revision to Diff 82986. alexshap added a comment. switch to --driver-mode=g++ Repository: rL LLVM https://reviews.llvm.org/D28269 Files: utils/perf-training/lit.cfg utils/perf-training/order-files.lit.cfg Index: utils/perf-training/order-files.lit.cfg

[PATCH] D28269: [clang] Update lit config in utils/perf-training

2017-01-03 Thread Alexander Shaposhnikov via Phabricator via cfe-commits
alexshap added a comment. btw: --driver-mode=g++ works as well Repository: rL LLVM https://reviews.llvm.org/D28269 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D28269: [clang] Update lit config in utils/perf-training

2017-01-03 Thread Alexander Shaposhnikov via Phabricator via cfe-commits
alexshap added a comment. it works for me, i have just double checked. Although to generate a file which provides any measurable improvements (was i measuring the total build time of LLVM (using the new binary)) i had to make some extra changes (on OSX 10.12.1) (slightly change dtrace script, r

[PATCH] D28269: [clang] Update lit config in utils/perf-training

2017-01-03 Thread Chris Bieneman via Phabricator via cfe-commits
beanz added a comment. I don't think this works either. Pretty sure not setting the driver mode defaults it to gcc, when we should be setting it to g++. Can you try setting driver mode to g++ instead of removing the flag? Repository: rL LLVM https://reviews.llvm.org/D28269 __

[PATCH] D28269: [clang] Update lit config in utils/perf-training

2017-01-03 Thread Alexander Shaposhnikov via Phabricator via cfe-commits
alexshap created this revision. alexshap added reviewers: mehdi_amini, beanz, bogner. alexshap added a subscriber: cfe-commits. alexshap set the repository for this revision to rL LLVM. This diff removes --driver-mode=cpp from utils/perf-training/order-files.lit.cfg and utils/perf-training/lit.cf