caml-list - the Caml user's mailing list
 help / color / mirror / Atom feed
From: Josh Berdine <josh@berdine.net>
To: caml users <caml-list@inria.fr>
Subject: Re: [Caml-list] Performance penalty for using monad
Date: Sun, 24 Sep 2017 21:04:56 +0100	[thread overview]
Message-ID: <33303E31-1874-4BD1-90D9-E9E8F623A6C6@berdine.net> (raw)
In-Reply-To: <CACLX4jR7JS=uOFsEhjDDHK76WBHh2NANT2dWs_-SaHmMTf4Zcg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1261 bytes --]

On Sep 24, 2017, at 12:28 PM, Yaron Minsky <yminsky@janestreet.com> wrote:
> For what it's worth, the main blocker for us turning Flambda on by default in classic mode is getting build artifact size and compilation speed down basically to the same level as closure-compilation. We're getting pretty close to that goal, though it will take a bit more time to get the improvements in question upstreamed.
> 
> So getting Flambda enabled by default isn't that far away (though most of the real benefits will require -O3, which will still lengthen compilation by quite a bit.)

Another data point on this: at facebook we recently switched the production infer binaries over to flambda -O3 (https://github.com/facebook/infer/commit/f8d7c810452ce3a4d2e7027e38f5d00426a2a917 <https://github.com/facebook/infer/commit/f8d7c810452ce3a4d2e7027e38f5d00426a2a917>). For local builds during development, we usually build without flambda, or actually even just bytecode. But for infer, flambda -O3 is worth 15-20% elapsed (~25% cpu) time, so it does not take an abnormal analysis run before that pays off the ~5x compile time deficit. (Given that we have to distribute a custom clang with the analyzer, build artifact size is basically in the noise.)

Cheers, Josh


[-- Attachment #2: Type: text/html, Size: 1868 bytes --]

  reply	other threads:[~2017-09-24 20:05 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-18 15:23 Helmut Brandl
2017-09-18 15:31 ` Yotam Barnoy
2017-09-19 12:46   ` Helmut Brandl
2017-09-19 13:24     ` Jesper Louis Andersen
2017-09-19 16:33       ` Helmut Brandl
2017-09-24  9:22         ` David Allsopp
2017-09-24 11:28           ` Yaron Minsky
2017-09-24 20:04             ` Josh Berdine [this message]
2017-09-25  8:32               ` Mark Shinwell
2017-09-25  9:25                 ` Josh Berdine

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=33303E31-1874-4BD1-90D9-E9E8F623A6C6@berdine.net \
    --to=josh@berdine.net \
    --cc=caml-list@inria.fr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).