Jason Gross (Jul 27 2022 at 15:03): Jason Gross (Jul 27 2022 at 15:03):I'm trying to figure out what's going wrong with https://github.com/mit-plv/fiat-crypto/pull/1350. src/PushButtonSynthesis/WordByWordMontgomeryReificationCache.v takes about twice as long to compile than it did before. Almost all the time is spent in Ltac that constructs constrs, so the Ltac profiler is not much help. I'm not really sure how to make sense of the data from perf record. (The change I made was to use transparent abstract to deduplicate reification work, so I'm guessing that either it's spending more time checking the cache (overhead would be underneath abstract/transparent_abstract), or looking up things in the cache (overhead would be underneath constr equality checking / match term with term), or maybe it's spending more time typechecking (overhead in constr:(...)) or creating / unifying / resolving evars. I don't know how to read this off the perf data, though...)
The slower version looks like
image.png
while the faster version looks like
image.png
(cc @Pierre-Marie Pédrot )
Janno (Jul 27 2022 at 15:07):I think the second screenshot is not a perf report, is it? Is that a mistake?
Jason Gross (Jul 27 2022 at 15:18):Oops, fixed, thanks @Janno
Janno (Jul 27 2022 at 15:24):It's hard to say what is going on in that perf profile with all these unknown functions. I am not sure what the reason for this is. The profile should have useful names for all functions in it.
Jason Gross (Jul 27 2022 at 15:36):Maybe this version of Coq or OCaml was somehow compiled without complete debug info?
Jason Gross (Jul 27 2022 at 15:44):objdump --syms $(which coqc) lists a bunch of things, but there are no symbols for ocamlc / ocamlopt
Jason Gross (Jul 27 2022 at 15:44):(This is on Arch)
Guillaume Melquiond (Jul 27 2022 at 15:56):This looks like bogus stack traces. Did you forget to pass --call-graph=dwarf to perf record?
Jason Gross (Jul 27 2022 at 22:09):Oh, I wasn't aware that I need to pass --call-graph=dwarf. I was just doing perf record -g
Jason Gross (Jul 27 2022 at 23:15):--call-graph=dwarf does not help
image.png
Pierre-Marie Pédrot (Jul 30 2022 at 16:52):@Jason Gross you need to use a version of the OCaml compiler with the fp flag for the callstack to be meaningful.
Pierre-Marie Pédrot (Jul 30 2022 at 16:52):I personally use ocaml-variants.4.09.1+fp on a daily basis.
Last updated: Apr 19 2024 at 13:02 UTC