[PATCH] D57835: Fix -ftime-report with -x ir

arsenm added a comment. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57835/new/ https://reviews.llvm.org/D57835 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D57835: Fix -ftime-report with -x ir

arsenm updated this revision to Diff 255095. arsenm added a comment. Rebase, add new PM run line CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57835/new/ https://reviews.llvm.org/D57835 Files: clang/lib/CodeGen/CodeGenAction.cpp clang/lib/Frontend/CompilerInstance.cpp clang/test/

[PATCH] D57835: Fix -ftime-report with -x ir

fedor.sergeev requested changes to this revision. fedor.sergeev added a comment. This revision now requires changes to proceed. just to make the comments visible :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57835/new/ https://reviews.llvm.org/D57835 __

[PATCH] D57835: Fix -ftime-report with -x ir

fedor.sergeev added inline comments. Comment at: test/Frontend/ftime-report-bitcode.ll:1 +; RUN: %clang_cc1 -triple x86_64-apple-darwin10 -ftime-report -emit-obj -o /dev/null %s 2>&1 | FileCheck %s + Can you, please, test -fexperimental-new-pass-manager mode as

[PATCH] D57835: Fix -ftime-report with -x ir

fedor.sergeev added a comment. In D57835#1387622 , @arsenm wrote: > I partially solved the problem with double printing the reports. It's from > this in cc1_main: > > // If any timers were active but haven't been destroyed yet, print theirp > // resu

[PATCH] D57835: Fix -ftime-report with -x ir

arsenm added a comment. Herald added a subscriber: jdoerfert. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57835/new/ https://reviews.llvm.org/D57835 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[PATCH] D57835: Fix -ftime-report with -x ir

arsenm added a comment. I partially solved the problem with double printing the reports. It's from this in cc1_main: // If any timers were active but haven't been destroyed yet, print theirp // results now. This happens in -disable-free mode. llvm::TimerGroup::printAll(llvm::errs()); On

[PATCH] D57835: Fix -ftime-report with -x ir

arsenm created this revision. Herald added a subscriber: wdng. This was only printing the clang frontend timer, and none of the backend timers. Set the llvm global for enabling the backend time report when creating the frontend timers, rather than in the BackendConsumer constructor. There