[PATCH] D63378: [ORC] WIP Speculative compilation

[Lang Hames via Phabricator via llvm-commits]

lhames added a comment.

Hi Praveen,

I'm not sure the IRSpeculator is necessary. My instinct would be to just have the Speculator class as the global repository of estimates of the form "execution point -> { candidates to speculatively compile }" (i.e. your GlobalSpecMap). Then the IRSpeculationLayer should take a reference to the Speculator and use a custom FunctionPass to populates the estimates.

I've attached an example of a hand rolled pass pipeline that should give you access to statically calculated block frequency estimates: F9502527: block-freq-pass-example.cpp <https://reviews.llvm.org/F9502527>

- Lang.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63378/new/

https://reviews.llvm.org/D63378