[PATCH] D52904: [hot-cold-split] fix static analysis of cold regions

[Sebastian Pop via Phabricator via llvm-commits]

sebpop added a comment.

> Are both fixes necessary to fix the issue (the one for back propagation and the one to bail out if the entry block is cold), or is either one sufficient? The patch description only mentions the former.

I am not sure whether the entry block will get marked as cold after the two fixes to the forward and back propagation.
We could try to transform the early return into an assert that the entry block of a function is never marked cold.
I will test this change.

Thanks for your careful review of the patch!


https://reviews.llvm.org/D52904