r339281 - [CodeGen][Timers] Enable llvm::TimePassesIsEnabled when -ftime-report is specified

[Hans Wennborg via cfe-commits]

Merged to 7.0 in r339341.

On Wed, Aug 8, 2018 at 9:14 PM, Craig Topper via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> Author: ctopper
> Date: Wed Aug  8 12:14:23 2018
> New Revision: 339281
>
> URL: http://llvm.org/viewvc/llvm-project?rev=339281&view=rev
> Log:
> [CodeGen][Timers] Enable llvm::TimePassesIsEnabled when -ftime-report is specified
>
> r330571 added a new FrontendTimesIsEnabled variable and replaced many usages of llvm::TimePassesIsEnabled. Including the place that set llvm::TimePassesIsEnabled for -ftime-report. The effect of this is that -ftime-report now only contains the timers specifically referenced in CodeGenAction.cpp and none of the timers in the backend.
>
> This commit adds back the assignment, but otherwise leaves everything else unchanged.
>
> Modified:
>     cfe/trunk/lib/CodeGen/CodeGenAction.cpp
>
> Modified: cfe/trunk/lib/CodeGen/CodeGenAction.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CodeGenAction.cpp?rev=339281&r1=339280&r2=339281&view=diff
> ==============================================================================
> --- cfe/trunk/lib/CodeGen/CodeGenAction.cpp (original)
> +++ cfe/trunk/lib/CodeGen/CodeGenAction.cpp Wed Aug  8 12:14:23 2018
> @@ -127,6 +127,7 @@ namespace clang {
>                                  CodeGenOpts, C, CoverageInfo)),
>            LinkModules(std::move(LinkModules)) {
>        FrontendTimesIsEnabled = TimePasses;
> +      llvm::TimePassesIsEnabled = TimePasses;
>      }
>      llvm::Module *getModule() const { return Gen->GetModule(); }
>      std::unique_ptr<llvm::Module> takeModule() {
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits