From: Mark Shinwell <mshinwell@janestreet.com>
To: Josh Berdine <josh@berdine.net>
Cc: caml users <caml-list@inria.fr>
Subject: Re: [Caml-list] Performance penalty for using monad
Date: Mon, 25 Sep 2017 09:32:27 +0100 [thread overview]
Message-ID: <CAM3Ki77cdyM20GD5D4+HH2OEkcbcU2Av6urVV0Woz5dNQGmfKA@mail.gmail.com> (raw)
In-Reply-To: <33303E31-1874-4BD1-90D9-E9E8F623A6C6@berdine.net>
[-- Attachment #1: Type: text/plain, Size: 1550 bytes --]
I would also point out that Facebook's experience with compilation speed
under Flambda is actually rather worse than what we've found at Jane
Street. We are typically seeing closer to 2.5x rather than 5x at -O3.
However we are working to reduce this further.
Mark
On 24 September 2017 at 21:04, Josh Berdine <josh@berdine.net> wrote:
> 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/f8d7c810452ce3a4d2e7027e38f5d0
> 0426a2a917). 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: 2327 bytes --]
next prev parent reply other threads:[~2017-09-25 8:33 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
2017-09-25 8:32 ` Mark Shinwell [this message]
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=CAM3Ki77cdyM20GD5D4+HH2OEkcbcU2Av6urVV0Woz5dNQGmfKA@mail.gmail.com \
--to=mshinwell@janestreet.com \
--cc=caml-list@inria.fr \
--cc=josh@berdine.net \
/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