I made the two tweaks suggested but as it turns out I can't commit to github, can someone land this for me?

Thanks Craig

Change unit test to run just prologepilog phase and use update_mir_test_checks.py

clang format

Add test case, thanks @jrtc27

An example of incorrect code in baseline is below where the spill of X1 is marked killed but is live to the following SW.

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: