Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

OlivierNicole

OlivierNicole created a comment on a pull request on ocaml/ocaml
Sorry for the delay. As this depends on #13438 to be useful, we should probably decide on the fate of that one first.

View on GitHub

OlivierNicole created a comment on a pull request on ocaml/ocaml
We could do that and option (3) above: > Add a warning to discourage it and decide that this PR, although it improves consistency, builds on top of a discouraged behaviour, and therefore should ...

View on GitHub

OlivierNicole created a comment on a pull request on ocsigen/js_of_ocaml
> Regarding the documentation, we should probably still document `--enable effects` somehow while dune has not been updated, which might take a while. I tried to hit a compromise, keeping the do...

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole pushed 2 commits to optim_effects OlivierNicole/js_of_ocaml
  • CR: simplify 17a2183
  • Docs: document double translation better Only document the use of `--enable effects` for now, as only this option is... 26c5fb8

View on GitHub

OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
It’s non-trivial because `Config.Flag.effects ()` is initialized by some separate logic, not in `Cmd_arg`. But I can try.

View on GitHub

OlivierNicole created a review on a pull request on ocsigen/js_of_ocaml

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml
  • Move assume_no_perform to Jsoo_runtime b7bcaa8

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml
  • Fix uses of `--enable=effects` in lib-wasm/ e52ab3a

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole pushed 59 commits to optim_effects OlivierNicole/js_of_ocaml
  • Test: global flow bug c0e3651
  • Global flow analysis: add some debugging code Make sure we have reached a fixed point. 18b1cbc
  • Global flow analysis: bug fix We were not correctly propagating the information that a block is possibly mutated. 71702fa
  • Modernize a bit Dom_html Mostly, removed opdef for properties that are widely available. eff5bac
  • CI updates (adapt Jane Street packages) dc6c457
  • dune: update menhir extension to 3.0 Signed-off-by: Sora Morimoto <[email protected]> 950b56a
  • Runtime: fix memory leak wrt channels 7309673
  • typo a79fb93
  • CI: fix effects with wasmoo 0b995da
  • Effects: double translation of functions and ... dynamic switching between direct-style and CPS code. (#1461) efcad4a
  • Add caml_assume_no_effects primitive and tests Passing a function [f] as argument of `caml_assume_no_effects` guaran... 08254f7
  • CR: Fix non-raised Unhandled with assume_no_perform 73d39ce
  • CR: Commit suggestion for compiler/lib/driver.ml Co-authored-by: Jérôme Vouillon <[email protected]> ce93d77
  • Fix runtime deps, add missing test ea71cf5
  • CR 4526fd9
  • CR b093844
  • Runtime format a9202c7
  • Factorize substitution logic ac1e088
  • CR: make caml_callback an alias of caml_call_gen... ... when double translation is enabled. b418cdf
  • Promote tests f2a00ee
  • and 39 more ...

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml
  • CR: simplify closure allocation 6b78233

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml
  • CR: add missing bound check 399bc41

View on GitHub

OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
Should I still apply `do_alloc_jump_closures` to both resulting lists?

View on GitHub

OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
Should I still apply `do_alloc_jump_closures` to both resulting lists?

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
I’m not sure to understand why Dune needs to know about Js_of_ocaml flags?

View on GitHub

OlivierNicole created a review on a pull request on ocsigen/js_of_ocaml

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml
  • Fix Dynlink and remove use of --enable=effects 1af64fb

View on GitHub

OlivierNicole pushed 49 commits to optim_effects OlivierNicole/js_of_ocaml
  • Misc: fix doc generation 0c19536
  • Misc: remove unused files e0ec46e
  • Build: dune alias for js and wasm (#1760) c0f9035
  • Runtime: no longer use arguments (#1740) * Runtime: no longer use arguments * no spread to build array * adapt... 145e4d2
  • Doc: document mininum requirements (#1762) 425df2e
  • Doc: document emcascript features we use (#1763) 25edc11
  • CI: use latest binaryen (#1767) Signed-off-by: Sora Morimoto <[email protected]> 4782b7b
  • Effects: double translation of functions and ... dynamic switching between direct-style and CPS code. (#1461) 018986c
  • Add caml_assume_no_effects primitive and tests Passing a function [f] as argument of `caml_assume_no_effects` guaran... 402ab87
  • CR: Fix non-raised Unhandled with assume_no_perform 25a2c9b
  • CR: Commit suggestion for compiler/lib/driver.ml Co-authored-by: Jérôme Vouillon <[email protected]> 3654c2b
  • Fix runtime deps, add missing test e0bd065
  • CR 18dc067
  • CR 04bfe84
  • Runtime format 617c65b
  • Factorize substitution logic f3a35b8
  • CR: make caml_callback an alias of caml_call_gen... ... when double translation is enabled. 77070a3
  • Promote tests 0abf21f
  • CR: Fix caml_trampoline_cps and rename it to caml_resume ca418fb
  • CR: First fixes to Lambda_lifting_simple b9af8b2
  • and 29 more ...

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole pushed 1 commit to optim_effects OlivierNicole/js_of_ocaml

View on GitHub

OlivierNicole created a review comment on a pull request on ocsigen/js_of_ocaml
Ah, I see.

View on GitHub

OlivierNicole created a review on a pull request on ocsigen/js_of_ocaml

View on GitHub

Load more