[PATCH] D77341: [DomTree] Replace ChildrenGetter with GraphTraits over GraphDiff.
Nikita Popov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 10 13:03:17 PDT 2020
nikic added a comment.
In D77341#2144974 <https://reviews.llvm.org/D77341#2144974>, @asbirlea wrote:
> Thank you for the testing. Could you help with with instructions on how to run the tracker myself?
> My local testing showed a negligible regression for mafft and a negligible improvement on other benchmarks, so it looked like noise on average.
The tracker just compiles test-suite under `perf stat` using the cached cmake configs. If I pick out the file with the largest regression (mafft `constants.c` in `ReleaseThinLTO` config with 18% regression) I can reproduce this locally as follows:
perf stat /home/nikic/llvm-project/build/bin/clang -DNDEBUG -O3 -fomit-frame-pointer -flto=thin -DNDEBUG -w -Werror=date-time -DLLVM -MD -MT MultiSource/Benchmarks/mafft/CMakeFiles/pairlocalalign.dir/constants.c.o -MF MultiSource/Benchmarks/mafft/CMakeFiles/pairlocalalign.dir/constants.c.o.d -o MultiSource/Benchmarks/mafft/CMakeFiles/pairlocalalign.dir/constants.c.o -c ../MultiSource/Benchmarks/mafft/constants.c
This gives me 3.5M instructions before and 4.2M instructions after. Those particular numbers are for an assertion-enabled build (the numbers on the website are without assertions.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77341/new/
https://reviews.llvm.org/D77341
More information about the cfe-commits
mailing list