[PATCH] D51467: Output per-function size-info remarks

[Jessica Paquette via Phabricator via llvm-commits]

paquette created this revision.
paquette added reviewers: anemet, thegameg.
Herald added a subscriber: mehdi_amini.

This patch adds per-function size information remarks. Previously, passing -Rpass-analysis=size-info would only give you per-module changes.

By adding the ability to do this per-function, it's easier to see which functions contributed the most to size changes.

(Note that we still have a random function name attached to the remark which is kind of misleading. Since we want to emit remarks for deleted functions as well, we can't use the function we're emitting the remark for in the `OptimizationRemarkAnalysis`. I put a FIXME in for this, but if there's a better way around it, it'd be nice to avoid the issue entirely.)


https://reviews.llvm.org/D51467

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51467.163226.patch
Type: text/x-patch
Size: 22423 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180829/7d692386/attachment.bin>