这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@smjleo
Copy link
Contributor

@smjleo smjleo commented Aug 29, 2025

I think they are small enough that it's unnecessary to separate them out into different PRs. The changes are:

  • remove unneeded ignore
  • avoid evaluating body twice for Rec_info
  • handle the non-effect None case for Apply_expr (some direct function calls in classic mode may omit the callee)

@smjleo smjleo requested a review from jvanburen August 29, 2025 21:37
@smjleo smjleo added the ocamlj Related to ocamlj/js_of_ocaml/JSIR label Aug 29, 2025
Base automatically changed from selee.jsir-produce-cmjx to selee.jsir-from-flambda2 September 3, 2025 14:48
@smjleo smjleo merged commit 8d47e5d into selee.jsir-from-flambda2 Sep 3, 2025
23 checks passed
@smjleo smjleo deleted the selee.jsir-misc-bugfix branch September 3, 2025 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ocamlj Related to ocamlj/js_of_ocaml/JSIR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants