[llvm] r200034 - Add Constant Hoisting Pass

[Juergen Ributzka]

Hi Hans,

there is a default implementation and the targets are free to override them if they have a better/more accurate cost model. If you look i.e. for the original getIntImmCost method, then you will see that only the ARM and the X86 target provide customized implementations. All other targets just use the default one.

I didn’t see the warning on my build, but I will look into them.

Cheers,
Juergen

On Jan 24, 2014, at 5:24 PM, Hans Wennborg <hans at chromium.org> wrote:

> I've reverted the patch in r200058. If you think that's wrong, feel
> free to re-apply it, but it seems weird to me that we're adding two
> new getIntImmCost overloads, and not overriding them in the non-X86
> targets.
> 
> Thanks,
> Hans