Cristiano Calcagno
Cristiano Calcagno
Going down the rabbit hole for a while, after confirming that the initial value of the global counter for the switch exit handlers determines which of the two possible forms...
Same comment: moved to the compiler. And there, some README is needed.
> @cristianoc any guess about the implementation? Not really but I'd start from the error message and add a few print statements until one finds the source.
> Maybe it's better to merge #6979 first, then I'll rebase Definitely. Given the size of this amazing work we can't really review it but we can quickly observe if...
Can't find the convo anymore. f == f being false has obvious footgun properties The other way is not perfect either Just saying: there are trade offs and one just...
@diml I'm testing out a global dead code analysis that relies on this. Currently I'm telling every user to use the workaround `dune build @check @all`. However, if there's a...
Hey @jeremiedimino thank you for the update. This is the current workaround I am suggesting: https://github.com/cristianoc/reanalyze#full-project-test-infer-native-project Once a better workaround has shipped, I will update the recommendation.
The ides is just to simplify the set-up instructions, so people can use their build with no changes. Not a huge deal, just a convenience.
> > When building the runtime with rewatch on Windows in CI, I get > > ``` > > Can't continue... Found a circular dependency in your code: > >...