Expand the simplification of special cases of log() to include log2() and log10() as well as intrinsics and more types.
Patch uploaded without context.
|1710 ↗||(On Diff #218803)|
Does this do the right thing for non-double libcalls?
|2888 ↗||(On Diff #218803)|
|93 ↗||(On Diff #218803)|
I'm not sure why you want to remove test coverage for log10f?
This should be using the overload of getLibFunc that takes a CallSite, instead of expanding it out by hand. This formulation skips checks that should happen otherwise (specifically, that it's not an indirect call, that the call isn't marked nobuiltin, and the function has an appropriate signature).
The CallSite version specifically checks that getCalledFunction doesn't return null. Because the case where it returns null is for an indirect call.