Ralender wrote: > This LGTM, should we update BlockFrequencyInfo too? yes BlockFrequencyInfo also gets outdated. but at least it doesn't crash. Also I am not sure it should be done here. https://github.com/llvm/llvm-project/pull/212058