[PATCH] D155688: [PATCH] [llvm] [InstCombine] Canonicalise ADD+GEP

2023-10-06 Thread Mats Petersson via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGe13bed4c5f35: [PATCH] [llvm] [InstCombine] Canonicalise ADD+GEP (authored by d-smirnov, committed by MatsPetersson). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-04-18 Thread Mats Petersson via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGefae695d52cc: Add -f[no-]loop-versioning option (authored by MatsPetersson). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION http

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-04-17 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 514525. Leporacanthicus added a comment. Rebase only Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.llvm.org/D141307 Files: clang/include/clang/Driver/Options.td clang/li

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-04-17 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 514403. Leporacanthicus added a comment. Rebase only Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.llvm.org/D141307 Files: clang/include/clang/Driver/Options.td clang/li

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-04-14 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 513672. Leporacanthicus added a comment. Remove unnecessary braces in if-statement. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.llvm.org/D141307 Files: clang/include/cla

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-04-12 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 512929. Leporacanthicus marked 3 inline comments as done. Leporacanthicus added a comment. - Add new function to help clarify the purpose of the options being used. - Add descriptive comment to make function use clearer. - Update tests to make them a m

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-04-11 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 512532. Leporacanthicus added a comment. Rebase only. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.llvm.org/D141307 Files: clang/include/clang/Driver/Options.td clang/l

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-04-06 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 511416. Leporacanthicus added a comment. Rebase only Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.llvm.org/D141307 Files: clang/include/clang/Driver/Options.td clang/li

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-04-04 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 510756. Leporacanthicus added a comment. Rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.llvm.org/D141307 Files: clang/include/clang/Driver/Options.td clang/lib/Dr

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-03-17 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 506125. Leporacanthicus added a comment. Rebase and fix conflicts Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.llvm.org/D141307 Files: clang/include/clang/Driver/Options.

[PATCH] D146075: [flang][driver][openmp] Write MLIR for -save-temps

2023-03-15 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus accepted this revision. Leporacanthicus added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146075/new/ https://reviews.llvm.org/D146075

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-03-08 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 503459. Leporacanthicus marked an inline comment as done. Leporacanthicus added a comment. Fix copied comment to reflect the new content. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https:

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-03-08 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus marked an inline comment as done. Leporacanthicus added inline comments. Comment at: clang/lib/Driver/ToolChains/Flang.cpp:69 + Arg *loopVersioning = + Args.getLastArg(options::OPT_Ofast, options::OPT_floop_versioning, + options::OPT_fno

[PATCH] D141307: Add -f[no-]loop-versioning option

2023-03-08 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 503360. Leporacanthicus added a comment. Herald added a reviewer: sscalpone. Updates based on review comments: - Add tests. - Enable on -O3 Also changed the name to match gfortran. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D141307: [WIP] Add -f[no-]loop-versioning option

2023-03-02 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 501881. Leporacanthicus added a comment. Update for rebase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.llvm.org/D141307 Files: clang/include/clang/Driver/Options.td c

[PATCH] D141307: [WIP] Add -f[no-]loop-versioning option

2023-02-16 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 498065. Leporacanthicus added a comment. Rebased and updated help-message Also using different type of template for the option Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.

[PATCH] D141307: [WIP] Add -f[no-]loop-versioning option

2023-01-13 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus updated this revision to Diff 488949. Leporacanthicus added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141307/new/ https://reviews.llvm.org/D141307 Files: clang/include/clang/Driver/Options.td clang/lib/Dri

[PATCH] D141307: [WIP] Add -f[no-]loop-versioning option

2023-01-09 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus created this revision. Herald added a reviewer: awarzynski. Herald added projects: Flang, All. Leporacanthicus requested review of this revision. Herald added subscribers: cfe-commits, jdoerfert, MaskRay. Herald added a project: clang. Posted to satisfy depenences of another patch.

[PATCH] D129156: Add -fpass-plugin option to Flang

2022-10-11 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus added a comment. > @MatsPetersson & @clementval , could you share you build command so that the > failure can be reproduced before this re-lands? I can't share the same command that I used, because it's some old CMAKE command that I no longer have in my history. But I tested thi

[PATCH] D129156: Add -fpass-plugin option to Flang

2022-10-05 Thread Mats Petersson via Phabricator via cfe-commits
MatsPetersson added a comment. In D129156#3837709 , @awarzynski wrote: > I suspect that this fails when running `ninja check-flang`, right? > > Most likely `Bye` needs to be added as a dependency for Flang tests, > something akin to this >

[PATCH] D129156: Add -fpass-plugin option to Flang

2022-10-05 Thread Mats Petersson via Phabricator via cfe-commits
MatsPetersson added a comment. In D129156#3836895 , @tarunprabhu wrote: > In D129156#3836838 , @clementval > wrote: > >> I probably didn't copy the error correctly .. this is the correct libs that >> is not fou

[PATCH] D122008: [flang][driver] Add support for generating executables

2022-04-01 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus accepted this revision. Leporacanthicus added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122008/new/ https://reviews.llvm.org/D122008 ___ cfe-commits mailing list cfe-c

[PATCH] D120246: [flang][driver] Add support for `--target`/`--triple`

2022-02-24 Thread Mats Petersson via Phabricator via cfe-commits
MatsPetersson added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120246/new/ https://reviews.llvm.org/D120246 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.o

[PATCH] D100558: [flang][driver] Add support for `-fget-definition`

2021-04-20 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus accepted this revision. Leporacanthicus added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100558/new/ https://reviews.llvm.org/D100558 _

[PATCH] D28080: [Docs][OpenCL] Added OpenCL feature description to user manual.

2016-12-23 Thread Mats Petersson via Phabricator via cfe-commits
Leporacanthicus added inline comments. Comment at: docs/UsersManual.rst:2083 +space is represented by the absence of an address space attribute in the IR (see +also :ref:`the section on address space attribute `). + Should this be `opencl_addrsp` (two d's)? ==