Skip to content

pre-commit: PRantoniofrighetto/llvm-project/pull/1 #1726

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

pre-commit: PRantoniofrighetto/llvm-project/pull/1 #1726

wants to merge 2 commits into from

Conversation

dtcxzyw
Copy link
Owner

@dtcxzyw dtcxzyw commented Nov 20, 2024

@github-actions github-actions bot mentioned this pull request Nov 20, 2024
Open
@github-actions GitHub Actions
Copy link
Contributor

runner: ariselab-64c
baseline runner: cseadmin-PowerEdge-R830
baseline: llvm/llvm-project@62bf584
patch: antoniofrighetto/llvm-project#1
sha256: e622a2ddcb1c63c5d4d530294dd849b60cc310f426323e905ca7c490a6a73020
commit: 709da61
173 files changed, 17282 insertions(+), 17368 deletions(-)

Compilation time result (by files):
Top 5 improvements:
  cmake/archive_write_set_format_cpio_binary.c.ll 139583887 -> 139247717 -0.24%
  openssl/libcrypto-lib-ocb128.ll 261814054 -> 261226380 -0.22%
  openssl/libcrypto-shlib-ocb128.ll 261682097 -> 261149724 -0.20%
  linux/netlabel.ll 171960285 -> 171614541 -0.20%
  linux/swnode.ll 410791361 -> 409971618 -0.20%
Top 5 regressions:
  abc/abcNpnSave.c.ll 290169754 -> 291247796 +0.37%
  abc/mapperTable.c.ll 150986072 -> 151539122 +0.37%
  abc/mapperCreate.c.ll 259030992 -> 259925503 +0.35%
  abc/hopTable.c.ll 149719679 -> 150230692 +0.34%
  abc/fraigTable.c.ll 297757420 -> 298565742 +0.27%

Overall: -0.04703204%

Compilation time result (by projects):
Top 5 improvements:
  cpp-httplib 18245304111 -> 18221033058 -0.13%
  cjson 1422515744 -> 1420701613 -0.13%
  libevent 11173382335 -> 11162020648 -0.10%
Top 5 regressions:

Overall: -0.05382046%

Improvements:
  instsimplify.NumReassoc 343220 -> 344396 +0.34%
  loop-instsimplify.NumSimplified 192659 -> 192743 +0.04%
  correlated-value-propagation.NumMinMax 9077 -> 9080 +0.03%
  correlated-value-propagation.NumCmps 241028 -> 241086 +0.02%
  indvars.NumReplaced 99167 -> 99184 +0.02%
  loop-delete.NumDeleted 143114 -> 143133 +0.01%
  gvn.NumGVNBlocks 208684 -> 208697 +0.01%
  correlated-value-propagation.NumSExt 36801 -> 36802 +0.00%
  function-attrs.NumWillReturn 1286267 -> 1286283 +0.00%
  memdep.NumCacheCompleteNonLocalPtr 4139983 -> 4140031 +0.00%
Regressions:
  simple-loop-unswitch.NumSelects 1872 -> 1861 -0.59%
  instcombine.NumConstProp 108097 -> 108068 -0.03%
  simple-loop-unswitch.NumBranches 82823 -> 82812 -0.01%
  correlated-value-propagation.NumSelects 209332 -> 209316 -0.01%
  adce.NumRemoved 91657 -> 91651 -0.01%
  dse.NumFastOther 129992 -> 129985 -0.01%
  loop-delete.NumBackedgesBroken 38734 -> 38732 -0.01%
  simplifycfg.NumSpeculations 314894 -> 314883 -0.00%
  instcombine.NumReassoc 254282 -> 254274 -0.00%
  correlated-value-propagation.NumAddNUW 159274 -> 159270 -0.00%

709da61 pre-commit: Update
41 41 bench/abc/optimized/Fxch.c.ll
20 20 bench/abc/optimized/abcAig.c.ll
50 50 bench/abc/optimized/abcNpnSave.c.ll
118 118 bench/abc/optimized/abcSymm.c.ll
105 105 bench/abc/optimized/absGlaOld.c.ll
23 23 bench/abc/optimized/dauTree.c.ll
33 33 bench/abc/optimized/extraUtilDsd.c.ll
30 30 bench/abc/optimized/fraigTable.c.ll
68 68 bench/abc/optimized/giaCut.c.ll
36 36 bench/abc/optimized/giaNf.c.ll
51 51 bench/abc/optimized/giaPat2.c.ll
4 4 bench/abc/optimized/giaTransduction.cpp.ll
225 225 bench/abc/optimized/giaTruth.c.ll
87 87 bench/abc/optimized/ifDsd.c.ll
58 58 bench/abc/optimized/ifMan.c.ll
57 57 bench/abc/optimized/ifTruth.c.ll
31 31 bench/abc/optimized/mpmPre.c.ll
3 3 bench/abc/optimized/satSolver.c.ll
211 211 bench/assimp/optimized/glTF2Importer.cpp.ll
38 38 bench/cvc5/optimized/cadical.cpp.ll
44 44 bench/delta-rs/optimized/4say4x9grcidoih4.ll
5 5 bench/draco/optimized/mesh_stripifier.cc.ll
132 132 bench/draco/optimized/sequential_integer_attribute_decoder.cc.ll
48 48 bench/gromacs/optimized/pme_load_balancing.cpp.ll
22 22 bench/hermes/optimized/CommandLine.cpp.ll
38 36 bench/libjpeg-turbo/optimized/jquant2.c.ll
9 12 bench/lief/optimized/ecp_curves.c.ll
11 11 bench/lightgbm/optimized/metadata.cpp.ll
47 47 bench/llvm/optimized/CommandLine.cpp.ll
2 1 bench/lodepng/optimized/lodepng.cpp.ll
216 213 bench/opencv/optimized/cap_mjpeg_encoder.cpp.ll
2 2 bench/opencv/optimized/fed.cpp.ll
144 144 bench/opencv/optimized/grfmt_exr.cpp.ll
103 103 bench/openexr/optimized/ImfRgbaFile.cpp.ll
56 57 bench/openjdk/optimized/abstractDisassembler.ll
39 37 bench/openjdk/optimized/jquant2.ll
240 243 bench/openssl/optimized/libcrypto-lib-ocb128.ll
25 25 bench/openssl/optimized/openssl-bin-s_server.ll
13 13 bench/postgres/optimized/geo_ops.ll
10 10 bench/proxygen/optimized/Logging.cpp.ll
26 25 bench/pybind11/optimized/test_numpy_dtypes.cpp.ll
59 59 bench/quantlib/optimized/sobolrsg.ll
6 6 bench/recastnavigation/optimized/NavMeshTesterTool.cpp.ll
174 176 bench/ruby/optimized/pack.ll
37 40 bench/slurm/optimized/xlate.ll
39 39 bench/softposit-rs/optimized/5az6c15ag5q4gib5.ll
96 92 bench/wireshark/optimized/msg_ulmap.c.ll
1 1 bench/wireshark/optimized/packet-iuup.c.ll
20 28 bench/z3/optimized/aig.cpp.ll
51 47 bench/z3/optimized/bv_solver.cpp.ll
7 8 bench/z3/optimized/euf_proof.cpp.ll
37 37 bench/z3/optimized/sat_binspr.cpp.ll
45 44 bench/z3/optimized/sat_local_search.cpp.ll
21 21 bench/z3/optimized/sat_solver.cpp.ll
9 9 bench/z3/optimized/smt_context_pp.cpp.ll
21 20 bench/z3/optimized/smt_internalizer.cpp.ll
26 26 bench/z3/optimized/theory_array.cpp.ll
83 80 bench/zxing/optimized/DMDetector.cpp.ll

antoniofrighetto
antoniofrighetto reviewed Nov 20, 2024
View reviewed changes
bench/lief/optimized/ecp_curves.c.ll
Comment on lines +3414 to +3415
%696 = and i64 %695, 1
%.not324 = icmp eq i64 %696, 0
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whole bunch of improvements.

bench/lodepng/optimized/lodepng.cpp.ll
Comment on lines +396 to +397
%and47.i = and i64 %inc43.i, 1
%tobool48.not.i = icmp eq i64 %and47.i, 0
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regression.

bench/openjdk/optimized/abstractDisassembler.ll
%.037.us = phi i32 [ %.0.us, %25 ], [ %.034, %.lr.ph38 ]
%.0.in36.us = phi i32 [ %.037.us, %25 ], [ %2, %.lr.ph38 ]
.lr.ph36.split.us: ; preds = %.lr.ph36, %26
%.035.us = phi i32 [ %.0.us, %26 ], [ %.033, %.lr.ph36 ]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Improvement.

bench/openjdk/optimized/jquant2.ll
Comment on lines +158 to +159
%84 = xor i32 %83, 1
%85 = add nuw nsw i32 %84, %.138.i
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regression.

bench/openssl/optimized/libcrypto-lib-ocb128.ll
Comment on lines +717 to +719
%i.0123 = phi i64 [ %i.0119, %for.body.lr.ph ], [ %i.0, %if.end22 ]
%in.addr.1122 = phi ptr [ %in, %for.body.lr.ph ], [ %add.ptr, %if.end22 ]
%out.addr.1121 = phi ptr [ %out, %for.body.lr.ph ], [ %add.ptr82, %if.end22 ]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Improvements (x2)

bench/pybind11/optimized/test_numpy_dtypes.cpp.ll
Comment on lines +69832 to +69833
%38 = trunc i64 %36 to i8
%39 = and i8 %38, 1
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regression.

bench/wireshark/optimized/msg_ulmap.c.ll
Comment on lines +3774 to +3776
%19 = and i32 %18, 1
%.not50 = icmp eq i32 %19, 0
br i1 %.not50, label %24, label %20
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regression.

bench/z3/optimized/aig.cpp.ll
Comment on lines +13048 to +13053
%xor.i90 = xor i64 %19, 1
%xor.i91 = xor i64 %8, 1
%25 = inttoptr i64 %xor.i91 to ptr
%cmp.i94.not.not = icmp eq i64 %and.i4.i83, 0
br i1 %cmp.i94.not.not, label %if.then63, label %if.else67
%and.i93 = and i64 %xor.i90, 1
%cmp.i94.not = icmp eq i64 %and.i93, 0
br i1 %cmp.i94.not, label %return.sink.split, label %if.then63
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whole bunch of improvements.

@antoniofrighetto
Copy link

/add-label reviewed

@antoniofrighetto antoniofrighetto mentioned this pull request Nov 20, 2024
Closed
@antoniofrighetto
Copy link

/close

@github-actions github-actions bot closed this Nov 27, 2024
@dtcxzyw dtcxzyw deleted the test-run11932097834 branch January 9, 2025 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@antoniofrighetto antoniofrighetto antoniofrighetto left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dtcxzyw @antoniofrighetto