Never inline intrinsic functions

[gptsarthak]

Fixes#1698

importtimeimportrandomimportmathprint(time.time())
print(random.randrange(10,20))
print(random.random())
print(math.atanh(0.90))

Main:

(lp) sarthak@pop-os:~/lpython$ lpython --fast examples/time.py0.00000000000000000e+0009.53282412436823800e-1309.53282412436823800e-130

On branch:

(lp) sarthak@pop-os:~/lpython_new/lpython$ src/bin/lpython --fast examples/hi.py1.68139046414960361e+09133.94382926819093038e-011.47221948958322035e+00

[gptsarthak]

There is no harm in doing this since all functions whose name starts with _lfortran_ are intrinsic.
This might just be a temporary fix, until the problem in #1698 (comment) is fixed.

[certik]

Thanks for the workaround!

I would rather fix this properly. The runtime library has a few hacks in the code base and this PR feels to me like it is adding another hack. I would rather remove some hacks, or add them into one place and wrap them in some functions.

If the problem is that we can't reliably identify intrinsic functions, then let's fix that.

[gptsarthak]

I guess we just have have to find a way so that functions that are coming from imported modules do not get inlined. @czgdp1807 do you know anything about that?

[czgdp1807]

I guess we just have have to find a way so that functions that are coming from imported modules do not get inlined.

Why so? If we are able to access the body of the function in ASR then I think inlining should be possible and we should do it. Probably that would be the proper fix instead of avoiding inlining them. Let's do it here and see what blocks us from properly inlining them. The only case which needs avoiding inlining is those functions which contain fallback implementations of optimisations such as flip_sign, etc, specifically, https://github.com/lfortran/lfortran/blob/main/src/runtime/builtin/lfortran_intrinsic_optimization.f90.

[gptsarthak]

But we cannot inline functions which are implemented in C using @ccall am I wrong?

[certik]

For BindC functions, we can inline the function call. Obviously we can't inline the function itself, since we don't have the source code.

[gptsarthak]

Is this alright? We stop inlining BindC function after we have inlined its fuction call.
--fast works alright with this fix.