{"id":2226773,"url":"http://patchwork.ozlabs.org/api/covers/2226773/?format=json","web_url":"http://patchwork.ozlabs.org/project/gcc/cover/bmm.hhup4p5wp2.gcc.gcc-TEST.pinskia.84.1.0@forge-stage.sourceware.org/","project":{"id":17,"url":"http://patchwork.ozlabs.org/api/projects/17/?format=json","name":"GNU Compiler Collection","link_name":"gcc","list_id":"gcc-patches.gcc.gnu.org","list_email":"gcc-patches@gcc.gnu.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"","list_archive_url":null,"date":"2026-04-22T18:49:08","name":"[v1,00/10] remove_fab","submitter":{"id":93219,"url":"http://patchwork.ozlabs.org/api/people/93219/?format=json","name":"Andrew Pinski via Sourceware Forge","email":"forge-bot+pinskia@forge-stage.sourceware.org"},"mbox":"http://patchwork.ozlabs.org/project/gcc/cover/bmm.hhup4p5wp2.gcc.gcc-TEST.pinskia.84.1.0@forge-stage.sourceware.org/mbox/","series":[{"id":501092,"url":"http://patchwork.ozlabs.org/api/series/501092/?format=json","web_url":"http://patchwork.ozlabs.org/project/gcc/list/?series=501092","date":"2026-04-22T18:49:11","name":"remove_fab","version":1,"mbox":"http://patchwork.ozlabs.org/series/501092/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/covers/2226773/comments/","headers":{"Return-Path":"","X-Original-To":["incoming@patchwork.ozlabs.org","gcc-patches@gcc.gnu.org"],"Delivered-To":["patchwork-incoming@legolas.ozlabs.org","gcc-patches@gcc.gnu.org"],"Authentication-Results":["legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=gcc.gnu.org\n (client-ip=2620:52:6:3111::32; helo=vm01.sourceware.org;\n envelope-from=gcc-patches-bounces~incoming=patchwork.ozlabs.org@gcc.gnu.org;\n receiver=patchwork.ozlabs.org)","sourceware.org; dmarc=none (p=none dis=none)\n header.from=forge-stage.sourceware.org","sourceware.org;\n spf=pass smtp.mailfrom=forge-stage.sourceware.org","server2.sourceware.org;\n arc=none smtp.remote-ip=38.145.34.39"],"Received":["from vm01.sourceware.org (vm01.sourceware.org\n [IPv6:2620:52:6:3111::32])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g18Zx33jCz1y2d\n\tfor ; Thu, 23 Apr 2026 05:35:37 +1000 (AEST)","from vm01.sourceware.org (localhost [127.0.0.1])\n\tby sourceware.org (Postfix) with ESMTP id 96EF2442B6DA\n\tfor ; Wed, 22 Apr 2026 19:35:35 +0000 (GMT)","from forge-stage.sourceware.org (vm08.sourceware.org [38.145.34.39])\n by sourceware.org (Postfix) with ESMTPS id A3F0840A8DD8\n for ; Wed, 22 Apr 2026 18:50:41 +0000 (GMT)","from forge-stage.sourceware.org (localhost [IPv6:::1])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n key-exchange x25519 server-signature ECDSA (prime256v1) server-digest SHA256)\n (No client certificate requested)\n by forge-stage.sourceware.org (Postfix) with ESMTPS id 705BC4358F\n for ; Wed, 22 Apr 2026 18:50:41 +0000 (UTC)"],"DKIM-Filter":["OpenDKIM Filter v2.11.0 sourceware.org 96EF2442B6DA","OpenDKIM Filter v2.11.0 sourceware.org A3F0840A8DD8"],"DMARC-Filter":"OpenDMARC Filter v1.4.2 sourceware.org A3F0840A8DD8","ARC-Filter":"OpenARC Filter v1.0.0 sourceware.org A3F0840A8DD8","ARC-Seal":"i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1776883841; cv=none;\n b=rJNguS4We31RnYgo5IIO1XhR5lzD8O2v1s//tCKeFAxT8VbBsmo0Ty06UfC3fcdFszRkY9K5FlQUh8YbD/reDQwh7K8uA+z5AStKFRs3BDoqyKnqdctzpoJzzRg0Glh9IoOzcM/JzXh4ASbp9NzKR7yldHVqNwkwd/CVC340Jyg=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=sourceware.org; s=key;\n t=1776883841; c=relaxed/simple;\n bh=wnAPEjhGEfkwxZ14bV29GxFKgflQy6NK3QklSuEXL5Y=;\n h=From:Date:Subject:To:Message-ID;\n b=VgoNd1s0vRyLm+sUO6BBlyH14TCTWaCgfLOsduQm1u7yCIAuwlWlOXIS++m4fTkTtBUuo1oUa+o3ld1hYRM4uFJ3TGHrBSRAK92hvRQuxKVvq3VMq15J6/euDjIfpjDE8fRouPjxzk65hgCqgzYCU08GCr39tyxn+GXqC/YyIuo=","ARC-Authentication-Results":"i=1; server2.sourceware.org","From":"Andrew Pinski via Sourceware Forge\n ","Date":"Wed, 22 Apr 2026 18:49:08 +0000","Subject":"[PATCH v1 00/10] remove_fab","To":"gcc-patches mailing list ","Message-ID":"\n ","X-Mailer":"batrachomyomachia","X-Pull-Request-Organization":"gcc","X-Pull-Request-Repository":"gcc-TEST","X-Pull-Request":"https://forge.sourceware.org/gcc/gcc-TEST/pulls/84","X-BeenThere":"gcc-patches@gcc.gnu.org","X-Mailman-Version":"2.1.30","Precedence":"list","List-Id":"Gcc-patches mailing list ","List-Unsubscribe":",\n ","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n ","Reply-To":"gcc-patches mailing list ,\n pinskia@gcc.gnu.org","Errors-To":"gcc-patches-bounces~incoming=patchwork.ozlabs.org@gcc.gnu.org"},"content":"Hi gcc-patches mailing list,\nAndrew Pinski via Sourceware Forge has requested that the following forgejo pull request\nbe published on the mailing list.\n\nCreated on: 2025-09-23 20:46:14+00:00\nLatest update: 2025-10-03 15:37:02+00:00\nChanges: 25 changed files, 1418 additions, 1550 deletions\nHead revision: pinskia/gcc-TEST ref remove_fab commit 54194efe9062df3e7a0cd7c3c08dca0cad5f3f0a\nBase revision: gcc/gcc-TEST ref trunk commit fb0702c6bd2d265aed452ba96e639d0ec4847d4f r16-4136-gfb0702c6bd2d26\nMerge base: fb0702c6bd2d265aed452ba96e639d0ec4847d4f\nFull diff url: https://forge.sourceware.org/gcc/gcc-TEST/pulls/84.diff\nDiscussion: https://forge.sourceware.org/gcc/gcc-TEST/pulls/84\nRequested Reviewers:\n\nThis patch set is to remove the fold_builtins (fab) pass. This moves the functionality of the pass to other locations and then finally removes the pass and for -Og replace it and the copy-prop pass in the pipeline with a forwprop pass.\nSome of the changes are cleanups before the move of that optimization.\nHere are where each of the optimization go:\nmemcmp -> forwprop\nASSUME -> gimple fold\nstack restore -> forwprop\nunreachable -> forwprop\nva_args_* -> gimple fold\natomic_*/sync_* -> isel\n\n\nChanged files:\n- A: gcc/testsuite/gcc.dg/tree-ssa/pr122033-1.c\n- A: gcc/testsuite/gcc.dg/tree-ssa/pr122033-2.c\n- M: gcc/doc/passes.texi\n- M: gcc/gimple-fold.cc\n- M: gcc/gimple-isel.cc\n- M: gcc/passes.def\n- M: gcc/testsuite/gcc.dg/builtin-unreachable-5.c\n- M: gcc/testsuite/gcc.dg/builtin-unreachable-6.c\n- M: gcc/testsuite/gcc.dg/builtin-unreachable-6a.c\n- M: gcc/testsuite/gcc.dg/builtin-unreachable-7.c\n- M: gcc/testsuite/gcc.dg/pr78408-2.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/builtin-fprintf-1.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/builtin-fprintf-chk-1.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/builtin-printf-1.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/builtin-printf-chk-1.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/builtin-vfprintf-1.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/builtin-vfprintf-chk-1.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/builtin-vprintf-1.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/builtin-vprintf-chk-1.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/pr79691.c\n- M: gcc/testsuite/gcc.dg/tree-ssa/ssa-ccp-10.c\n- M: gcc/tree-pass.h\n- M: gcc/tree-ssa-ccp.cc\n- M: gcc/tree-ssa-forwprop.cc\n- M: gcc/tree.h\n\n\nAndrew Pinski (10):\n fab/forwprop: Move memcmp->memcmp_eq to forwprop\n fab: Manaully build gimple rather than depend on gimplifier for stdarg\n functions\n fab: rewrite optimize_stack_restore call check [PR122033]\n fab/gimple-fold: Move removal of ASSUME internal function to gimple\n fold [PR121762]\n fab/forwprop: Move optimize stack restore to forwprop [PR121762]\n fab/forwprop: Move optimize_unreachable to forwprop [PR121762]\n fab/gimple-fold/forwprop: Move va_args folding to gimple_fold\n [PR121762]\n fab: Use a macro for the atomic/sync builtins case\n fab/isel: Move atomic optimizations to isel from fab [PR121762]\n Remove fold_builtin pass [PR121762]\n\n gcc/doc/passes.texi | 6 -\n gcc/gimple-fold.cc | 131 ++\n gcc/gimple-isel.cc | 956 +++++++++++\n gcc/passes.def | 8 +-\n gcc/testsuite/gcc.dg/builtin-unreachable-5.c | 10 +-\n gcc/testsuite/gcc.dg/builtin-unreachable-6.c | 6 +-\n gcc/testsuite/gcc.dg/builtin-unreachable-6a.c | 6 +-\n gcc/testsuite/gcc.dg/builtin-unreachable-7.c | 8 +-\n gcc/testsuite/gcc.dg/pr78408-2.c | 4 +-\n .../gcc.dg/tree-ssa/builtin-fprintf-1.c | 20 +-\n .../gcc.dg/tree-ssa/builtin-fprintf-chk-1.c | 20 +-\n .../gcc.dg/tree-ssa/builtin-printf-1.c | 22 +-\n .../gcc.dg/tree-ssa/builtin-printf-chk-1.c | 22 +-\n .../gcc.dg/tree-ssa/builtin-vfprintf-1.c | 16 +-\n .../gcc.dg/tree-ssa/builtin-vfprintf-chk-1.c | 16 +-\n .../gcc.dg/tree-ssa/builtin-vprintf-1.c | 16 +-\n .../gcc.dg/tree-ssa/builtin-vprintf-chk-1.c | 16 +-\n gcc/testsuite/gcc.dg/tree-ssa/pr122033-1.c | 18 +\n gcc/testsuite/gcc.dg/tree-ssa/pr122033-2.c | 23 +\n gcc/testsuite/gcc.dg/tree-ssa/pr79691.c | 2 +-\n gcc/testsuite/gcc.dg/tree-ssa/ssa-ccp-10.c | 4 +-\n gcc/tree-pass.h | 1 -\n gcc/tree-ssa-ccp.cc | 1442 -----------------\n gcc/tree-ssa-forwprop.cc | 186 ++-\n gcc/tree.h | 9 +\n 25 files changed, 1418 insertions(+), 1550 deletions(-)\n create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/pr122033-1.c\n create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/pr122033-2.c"}