Well, from my perspective, the test logic for this feature seems a bit odd:
even if you directly modify the function body to return NULL; or return 1/0;,
none of this execution logic takes effect at all—it serves no purpose whatsoever as long as inlining succeeds.
This could be unfriendly to developers who are new to this feature,
so I added a test for this scenario to verify that our logic falls back to the function body when inlining fails.
This may just be my personal opinion, though; if you think everything is fine, we can leave it as it is.
--
Regards,
Man Zeng
www.openhalo.org