[PATCH] D42212: [ThinLTO] Add call edges' relative block frequency to per-module summary.

Easwaran Raman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 24 15:17:04 PST 2018


eraman reopened this revision.
eraman added a comment.
This revision is now accepted and ready to land.

The patch caused two issues

- I used CI (of type CallInst*) to get the parent BB, but CI could be null. The fix is to use  &BB directly.
- Use of HotnessType Hotness : 3 gave warnings.  The fix is to make it uint32_t Hotness : 3 and use static_cast to convert back and forth from enum.

I will update the patch. PTAL.


Repository:
  rL LLVM

https://reviews.llvm.org/D42212





More information about the llvm-commits mailing list