[PATCH] D51276: [New PM][PassTiming] implement -time-passes for the new pass manager

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev updated this revision to Diff 167928.
fedor.sergeev added a comment.

running time-passes on longer pipelines revealed a problem with
making Timer managed by DenseMap in TimingData - we cant copy the Timer
on map expansion. Thus reverting back to keeping Timer* in that map
and new/deleting the instances.

Added to time-passes test an invocation that runs -O2 to expose this
situation (that particular invocation was failing in Timer's copy constructor
w/o the fix).


Repository:
  rL LLVM

https://reviews.llvm.org/D51276

Files:
  include/llvm/IR/PassTimingInfo.h
  include/llvm/Passes/StandardInstrumentations.h
  lib/IR/PassTimingInfo.cpp
  lib/Passes/StandardInstrumentations.cpp
  test/Other/time-passes.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51276.167928.patch
Type: text/x-patch
Size: 12659 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181002/1526c6ff/attachment-0001.bin>