Fixing & for C backend #1057
Fixing & for C backend #1057
Conversation
Signed-off-by: Ron Nuriel <[email protected]>
Signed-off-by: Ron Nuriel <[email protected]>
certik
force-pushed
the
feature/BitAnd_op
branch
from
f21ddd7 to
693ed9f
Compare
|
@ronnuriel I merged the first two commits, and also added a new commit (693ed9f) that updates the reference tests, I just did: and committed. |
| @@ -4,13 +4,23 @@ | |||
| def add(x: i32, y: i32) -> i32: | |||
| return x + y | |||
|
|
|||
| @inline | |||
| def and_op(x: i32, y: i32) -> i32: | |||
There was a problem hiding this comment.
I see. I think this feature isn't available in LLVM backend yet (will confirm though). I will add the same (after confirming).
There was a problem hiding this comment.
Go for it. Tests pass, but I assume LLVM ignores it.
There was a problem hiding this comment.
Well I am wondering if we should use the function inline pass for this. A function will be inlined in LLVM backend if it's decorated with @inline. Make sense?
In --fast flag all functions (provided they satisfy some conditions) are anyways inlined automatically.
There was a problem hiding this comment.
Yes. LLVM ignores this decorator. Making it work via function inline pass.
Signed-off-by: Ron Nuriel [email protected]