copy & paste & modify :-) from Mehdi pointerwd

cleanup

use CallBase

remove facebook markers

This change breaks builds fort compilers because the "&&" in the preprocessor does not protect the case that __has_cpp_attribute is not defined. Better something like

fix formatting

apply suggestion from @vsk

address comments

simplify loop continue test

Resetting the function map is a small (< 2%) win

clear map for every function

refactor common code, update test

add test

remove some unneeded changes, refactor location of updateProfileData, now updateProfileCallee

one hash lookup

The first change is simply performance related.
The second allows a client to incrementally build a vector, using the getMutableFrame method to access individual frames

refactor and update

Sorry for the delay.
Refactored the code in InlineFunction for reuse and to update weights uniformly.

use mutable keyword

Thanks for the review.
This was intended to balance the function updateCalleeCount in InlineFunction.cpp which simply decrements the entry count.

fix typo

ping

thanks for the suggestions

refine test

address review comments

remove facebook marker

Hi Xin, let me make sure I understand given:

remove unused declarations

use existing inverse_post_order

Use reverse CFG numbering to alter control flow

Regarding the question of simpler update to the control flow, I reviewed two references from @dberlin:

reduce scope of NewBranches, fix formatting

reduce scope of NewBranches, fix formatting

ping

Use depth_first_ext() template

ping