Skip to content

Conversation

zyw-bot
Copy link
Collaborator

@zyw-bot zyw-bot commented Aug 4, 2025

Link: llvm/llvm-project#151961
Requested by: @andjo403

@github-actions github-actions bot mentioned this pull request Aug 4, 2025
@zyw-bot
Copy link
Collaborator Author

zyw-bot commented Aug 4, 2025

Diff mode

runner: ariselab-64c-docker
baseline: llvm/llvm-project@da11c1d
patch: llvm/llvm-project#151961
sha256: d9a61419a129c254dbbbb77c18201e74b81151e6099036c6b9b46a15958293b7
commit: 7c46421

71 files changed, 27859 insertions(+), 28745 deletions(-)

Improvements:
  instcombine.NumDeadStore 26895 -> 26938 +0.16%
  memdep.NumCacheNonLocal 21176 -> 21177 +0.00%
  loop-delete.NumBackedgesBroken 45592 -> 45594 +0.00%
  local.NumPHICSEs 198239 -> 198247 +0.00%
  memdep.NumUncacheNonLocal 26584 -> 26585 +0.00%
  simplifycfg.NumSinkCommonCode 373207 -> 373218 +0.00%
  instcombine.NumConstProp 157928 -> 157932 +0.00%
  instcombine.NumPHICSEs 2388880 -> 2388938 +0.00%
  licm.NumSunk 267560 -> 267566 +0.00%
  simplifycfg.NumSinkCommonInstrs 807241 -> 807253 +0.00%
Regressions:
  loop-simplify.NumNested 12200 -> 12198 -0.02%
  adce.NumBranchesRemoved 23404 -> 23402 -0.01%
  adce.NumRemoved 99720 -> 99712 -0.01%
  gvn.NumGVNSimpl 4587342 -> 4587033 -0.01%
  instsimplify.NumSimplified 2530238 -> 2530068 -0.01%
  lcssa.NumLCSSA 15586534 -> 15585622 -0.01%
  instcombine.NegatorMaxDepthVisited 20605 -> 20604 -0.00%
  bdce.NumRemoved 386796 -> 386782 -0.00%
  correlated-value-propagation.NumPhis 1296381 -> 1296349 -0.00%
  correlated-value-propagation.NumPhiCommon 56346 -> 56345 -0.00%

wasmtime-rs/4ab4rlryc5h7bf6z.ll fail

5 11 bench/meilisearch-rs/optimized/2bqmf34tdvo23w8l.ll
7 7 bench/pola-rs/optimized/9q9n1s2mf6dwmafgbo3deymin.ll
6 8 bench/ruff-rs/optimized/8dg5gv1ul0w7vccunqd3ii3jp.ll
23 35 bench/uv-rs/optimized/02j162ogfl3swyferjwc5z24s.ll
26 38 bench/uv-rs/optimized/02tj2fo1cj1tiawt8ok7oppte.ll
8 18 bench/uv-rs/optimized/0h9u9hk1ysy5my10ukpk1cnd6.ll
32 44 bench/uv-rs/optimized/0qlsp9qlwvcslghwc8x9uqznx.ll
7 17 bench/uv-rs/optimized/18arzh82kngvwog07cz62qmei.ll
10 19 bench/uv-rs/optimized/1ffznczu4pieguhghdu5v078n.ll
52 65 bench/uv-rs/optimized/2rwzo8eniyqblpi6p2dviozdb.ll
9 20 bench/uv-rs/optimized/3us9u5hck9f6xy9154pes5hh5.ll
9 15 bench/uv-rs/optimized/70oya9zcywt1h05ti94rztobj.ll
25 31 bench/uv-rs/optimized/7z8izwoagwu7z9lvjzdkxy4q6.ll
7 13 bench/uv-rs/optimized/8kj46wae97fe0j9anf7v7m8mh.ll
6 10 bench/uv-rs/optimized/946sogpeyle3g2ctmg1pmagjd.ll
28 44 bench/uv-rs/optimized/bi4c58bghet8qnxsc146d76yy.ll
11 24 bench/uv-rs/optimized/cpcs1za9hsjeowfvmgfmet1kp.ll
24 29 bench/uv-rs/optimized/cv2ddfaw9rv2rrwcb1ub9a7jl.ll
5 12 bench/uv-rs/optimized/d0qnyr9gdf5wrlu00ph9493b9.ll
8 21 bench/uv-rs/optimized/de18et5bm77j9d1e3p5gzdl50.ll
24 30 bench/uv-rs/optimized/e6pgab6ywfjogigx9m8ep4647.ll
7 9 bench/wasmtime-rs/optimized/enal6epyb0tyurl.ll
2 2 bench/zed-rs/optimized/01hs87ka34wj50pyvose4nrzw.ll
8 18 bench/zed-rs/optimized/0drf4dow76rapncpvt6xcgeqh.ll
24 43 bench/zed-rs/optimized/1ftc77y2sv97gmniy36ag141f.ll
57 115 bench/zed-rs/optimized/1jfwd31zu9mxnid4fbavxtsbx.ll
10 20 bench/zed-rs/optimized/1z9x99vx9jfns7mxpwa762fd3.ll
8 14 bench/zed-rs/optimized/2hcgat59uf5mvvzma6d9cb3g0.ll
19 34 bench/zed-rs/optimized/44h9cwz011g0eagyds8si48mi.ll
12 23 bench/zed-rs/optimized/468j8mrahlfb4zd02cozu1ma6.ll
1 1 bench/zed-rs/optimized/55fm2nmy36xzz9o0ck5kapatu.ll
33 38 bench/zed-rs/optimized/5vh1wf9hqnwdftlb6xe6c9c43.ll
7 17 bench/zed-rs/optimized/8tscdb01zcabwdnjfgov818gm.ll
9 16 bench/zed-rs/optimized/b0jacrvunbgse6y69p1vx4gkq.ll
13 23 bench/zed-rs/optimized/cogvsa3bjgiftuxgwhwbiuj8x.ll
10 16 bench/zed-rs/optimized/dqryhdkxpzkfeay9pjzpwnhia.ll
5 12 bench/zed-rs/optimized/e80b5h5uaylvf0myf6av6e6rx.ll
24 34 bench/zed-rs/optimized/eiuikpvv7yixnsj9o23gd5xz0.ll
12 29 bench/zed-rs/optimized/f2m41hcwghjno5p8tkrposn1f.ll

Copy link
Contributor

github-actions bot commented Aug 4, 2025

The provided patch contains multiple changes across several benchmarks, primarily focused on improving control flow and eliminating unnecessary instructions. Here are the major changes:

  1. Control Flow Simplification: Several instances of redundant control flow, particularly involving CharSearcher and ReverseSearcher, have been simplified. For example, in uv-rs, the br instruction in memchr functions now directly branches to .loopexit instead of an intermediate block, reducing the number of basic blocks and improving clarity.

  2. Elimination of Redundant Phi Nodes: In multiple files (e.g., uv-rs, zed-rs), phi nodes that were previously used to handle undef values or redundant conditions have been removed or simplified. This includes replacing complex phi nodes with direct values or simpler expressions, reducing complexity and potential for undefined behavior.

  3. Optimization of Loop Exit Paths: Loop exit paths have been streamlined. For instance, in uv-rs, the .loopexit block no longer includes unnecessary trunc and br instructions, directly branching to the exit label. This reduces the number of instructions and improves performance.

  4. Simplification of Exception Handling: In ruff-rs, the handling of exception paths has been simplified by removing redundant checks and directly branching to the appropriate exit labels. This reduces the overhead of exception handling and makes the control flow more predictable.

  5. Reduction of Redundant Instructions: Various redundant instructions, such as unnecessary trunc and phi nodes, have been removed. For example, in wasmtime-rs, the %trunc.i.le and related instructions have been eliminated, simplifying the logic and reducing the instruction count.

These changes collectively aim to improve the efficiency and readability of the generated LLVM IR, reducing the number of instructions and simplifying control flow, which can lead to better performance and easier maintenance.

model: qwen-plus-latest
CompletionUsage(completion_tokens=393, prompt_tokens=118881, total_tokens=119274, completion_tokens_details=None, prompt_tokens_details=None)

@dtcxzyw
Copy link
Owner

dtcxzyw commented Aug 4, 2025

wasmtime-rs/4ab4rlryc5h7bf6z.ll fail

Unrelated crash. See llvm/llvm-project#128938 (comment).

@dtcxzyw dtcxzyw closed this Aug 5, 2025
@dtcxzyw dtcxzyw deleted the test-run16724613866 branch August 6, 2025 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants