[PATCH] D38768: Add remarks describing when a pass changes the IR instruction count of a module

[Jessica Paquette via Phabricator via llvm-commits]

paquette updated this revision to Diff 118984.
paquette added a comment.

Updated diff to reflect current comments on the review.

Highlights:

- Instruction counts are now only updated when there has *definitely* been a change.
- Instruction counts are now computed using deltas on smaller IR unit sizes whenever possible instead of on the entire module size. This will allow us to avoid recomputing module sizes whenever possible.
- Moved `getModuleInstrCount` into Module and introduced an equivalent function into Function.
- Expanded test so that it checks remarks for Module, Function, CGSCC, and BasicBlock passes.


https://reviews.llvm.org/D38768

Files:
  include/llvm/IR/Function.h
  include/llvm/IR/LegacyPassManagers.h
  include/llvm/IR/Module.h
  lib/Analysis/CallGraphSCCPass.cpp
  lib/IR/Function.cpp
  lib/IR/LegacyPassManager.cpp
  lib/IR/Module.cpp
  test/Other/size-remarks.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38768.118984.patch
Type: text/x-patch
Size: 22914 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171013/c21e2f57/attachment.bin>