From: Oliver S. <sc...@rb...> - 2012-07-19 08:29:43
|
Hi, I just noticed that there is a bug in the support for the native OCaml tools in one of the makefiles. The variables which hold the tool command names are defined as recursively expanded variables but they should be defined as simply expanded variables ('=' vs ':=' definition). The recursive variable flavor is not needed here and it produces a recursive definition error if NATIVECAML is defined for compilation. This is due to the recursive redefinition of CAMLDEP in this case. I attached a patch which replaces the variable definition type. Cheers Oliver 14.07.2012 08:41, Gabriel Kerneis: > On Fri, Jul 13, 2012 at 01:49:39PM +0200, Oliver Schwahn wrote: >> I just realized that the CIL build chain does not support the native >> versions of the OCaml tools. The native tool versions often >> outperform the byte code versions, e.g. ocamlc.opt (the native >> version) can compile CIL much faster than ocamlc (the byte code >> version) can. > Right, I used to have a local patch for this, but I never committed it. Your > patch is cleaner anyway, with proper configure checks. > >> I attached a patch for my modifications. Feel free to use it if it's >> of any use for you. > Applied, many thanks. |