[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

https://github.com/sebpop updated https://github.com/llvm/llvm-project/pull/142686 >From ff88172ea86363bd25f73fc78af1b8fca2cc4f3d Mon Sep 17 00:00:00 2001 From: Sebastian Pop Date: Thu, 22 May 2025 13:50:38 + Subject: [PATCH 1/2] add -floop-fuse to clang and flang --- clang/include/clang/

[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

https://github.com/nikic requested changes to this pull request. I don't think we should expose clang driver options for passes that are known to have significant issues. If you want to add a `cl::opt` flag to allow scheduling this in the pipeline and accessible for early testing via `-mllvm`,

[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

https://github.com/tblah approved this pull request. Flang changes LGTM. nit: I am no clang expert but I wonder if you could use MarshallingInfoFlag in Options.td to avoid having to set CGOpts.FuseLoops manually for clang (this is not supported for flang unfortunately). If people more familiar

[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

sebpop wrote: The goal is to make it easier to turn on loop fusion and to collect and fix bug reports against loop fusion. Once we are happy with the stability of the pass the goal is to enable by default in flang at -O2 and above. GCC does not have an equivalent for -floop-fuse. https://gi

[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Sebastian Pop (sebpop) Changes This adds the flag -floop-fuse to the clang and flang drivers. --- Full diff: https://github.com/llvm/llvm-project/pull/142686.diff 15 Files Affected: - (modified) clang/include/clang/Basic/CodeGenOptions

[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff HEAD~1 HEAD --extensions c,h,cpp -- clang/lib/CodeGen/BackendUtil.cpp clang/lib/Drive

[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

https://github.com/sebpop updated https://github.com/llvm/llvm-project/pull/142686 >From ff88172ea86363bd25f73fc78af1b8fca2cc4f3d Mon Sep 17 00:00:00 2001 From: Sebastian Pop Date: Thu, 22 May 2025 13:50:38 + Subject: [PATCH] add -floop-fuse to clang and flang --- clang/include/clang/Basi

[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

efriedma-quic wrote: If the goal here is gcc compatibility, I'd suggest not hooking up the flag to anything; the existing LoopFusePass isn't used by anything and hasn't been touched in years, so it's very likely to have issues. https://github.com/llvm/llvm-project/pull/142686 _

[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

llvmbot wrote: @llvm/pr-subscribers-clang-driver Author: Sebastian Pop (sebpop) Changes This adds the flag -floop-fuse to the clang and flang drivers. --- Full diff: https://github.com/llvm/llvm-project/pull/142686.diff 15 Files Affected: - (modified) clang/include/clang/Basic/CodeGen

[clang] [flang] [llvm] add -floop-fuse to clang and flang (PR #142686)

https://github.com/sebpop created https://github.com/llvm/llvm-project/pull/142686 This adds the flag -floop-fuse to the clang and flang drivers. >From d31011703460db137e050906de501a29c4c24fd5 Mon Sep 17 00:00:00 2001 From: Sebastian Pop Date: Thu, 22 May 2025 13:50:38 + Subject: [PATCH]