[libunwind] [OpenMPIRBuilder] Remove wrapper function in `createTask`, `createTeams` (PR #67723)

2023-10-05 Thread Kiran Chandramohan via cfe-commits
@@ -340,6 +340,44 @@ BasicBlock *llvm::splitBBWithSuffix(IRBuilderBase &Builder, bool CreateBranch, return splitBB(Builder, CreateBranch, Old->getName() + Suffix); } +// This function creates a fake integer value and a fake use for the integer +// value. It returns the fake

[clang] [OpenMPIRBuilder] Remove wrapper function in `createTask`, `createTeams` (PR #67723)

2023-10-05 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/67723 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] fc43c4f - [Flang] Include logical default with default-integer-8

2023-07-18 Thread Kiran Chandramohan via cfe-commits
Author: Kiran Chandramohan Date: 2023-07-18T11:05:25Z New Revision: fc43c4f0181bfb7e7821e8b12fbd45e5178e884b URL: https://github.com/llvm/llvm-project/commit/fc43c4f0181bfb7e7821e8b12fbd45e5178e884b DIFF: https://github.com/llvm/llvm-project/commit/fc43c4f0181bfb7e7821e8b12fbd45e5178e884b.diff

[clang-tools-extra] [OpenMPIRBuilder] Remove wrapper function in `createTask`, `createTeams` (PR #67723)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. Thanks for the detailed summary. LG. https://github.com/llvm/llvm-project/pull/67723 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinf

[libunwind] [OpenMPIRBuilder] Remove wrapper function in `createTask`, `createTeams` (PR #67723)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. Thanks for the detailed summary. LG. https://github.com/llvm/llvm-project/pull/67723 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinf

[clang] [OpenMPIRBuilder] Remove wrapper function in `createTask`, `createTeams` (PR #67723)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. Thanks for the detailed summary. LG. https://github.com/llvm/llvm-project/pull/67723 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinf

[libunwind] [OpenMPIRBuilder] Add ThreadLimit and NumTeamsUpper clauses to teams clause (PR #68364)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/68364 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libunwind] [OpenMPIRBuilder] Add ThreadLimit and NumTeamsUpper clauses to teams clause (PR #68364)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
@@ -1917,8 +1917,13 @@ class OpenMPIRBuilder { /// /// \param Loc The location where the teams construct was encountered. /// \param BodyGenCB Callback that will generate the region code. + /// \param NumTeamsUpper Upper bound on the number of teams. + /// \param Thread

[libunwind] [OpenMPIRBuilder] Add ThreadLimit and NumTeamsUpper clauses to teams clause (PR #68364)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan commented: Is this change for OMP 5.0 and before? https://github.com/llvm/llvm-project/pull/68364 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [OpenMPIRBuilder] Add ThreadLimit and NumTeamsUpper clauses to teams clause (PR #68364)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
@@ -1917,8 +1917,13 @@ class OpenMPIRBuilder { /// /// \param Loc The location where the teams construct was encountered. /// \param BodyGenCB Callback that will generate the region code. + /// \param NumTeamsUpper Upper bound on the number of teams. + /// \param Thread

[clang-tools-extra] [OpenMPIRBuilder] Add ThreadLimit and NumTeamsUpper clauses to teams clause (PR #68364)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan commented: Is this change for OMP 5.0 and before? https://github.com/llvm/llvm-project/pull/68364 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [OpenMPIRBuilder] Add ThreadLimit and NumTeamsUpper clauses to teams clause (PR #68364)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/68364 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [OpenMPIRBuilder] Add ThreadLimit and NumTeamsUpper clauses to teams clause (PR #68364)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
@@ -1917,8 +1917,13 @@ class OpenMPIRBuilder { /// /// \param Loc The location where the teams construct was encountered. /// \param BodyGenCB Callback that will generate the region code. + /// \param NumTeamsUpper Upper bound on the number of teams. + /// \param Thread

[clang-tools-extra] [mlir][OpenMP] Added omp.region operation (PR #65243)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: Can this have `omp.yield` as the terminator and be capable of yielding/returning values? https://github.com/llvm/llvm-project/pull/65243 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/ma

[clang] [mlir][OpenMP] Added omp.region operation (PR #65243)

2023-10-09 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: Can this have `omp.yield` as the terminator and be capable of yielding/returning values? https://github.com/llvm/llvm-project/pull/65243 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/ma

[clang] [flang] Add flags controlling whether to run the fir alias tags pass (PR #68595)

2023-10-10 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: Does `falias-analysis` control the existing TBAA generation as well? https://github.com/llvm/llvm-project/pull/68595 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co

[clang] [flang] add tbaa tags to global variables (PR #68727)

2023-10-11 Thread Kiran Chandramohan via cfe-commits
@@ -406,7 +406,7 @@ AliasAnalysis::Source AliasAnalysis::getSource(mlir::Value v) { attributes.set(Attribute::Pointer); } - if (type == SourceKind::Global) + if (type == SourceKind::Global || type == SourceKind::Direct) kiranchandramohan wrote:

[clang] [flang] add tbaa tags to global variables (PR #68727)

2023-10-11 Thread Kiran Chandramohan via cfe-commits
@@ -406,7 +406,7 @@ AliasAnalysis::Source AliasAnalysis::getSource(mlir::Value v) { attributes.set(Attribute::Pointer); } - if (type == SourceKind::Global) + if (type == SourceKind::Global || type == SourceKind::Direct) kiranchandramohan wrote:

[clang] [mlir][OpenMP] Added `omp.structured_region` operation (PR #68825)

2023-10-12 Thread Kiran Chandramohan via cfe-commits
@@ -1468,6 +1468,18 @@ LogicalResult CancellationPointOp::verify() { return success(); } +//===--===// +// RegionOp +//===--===// + +Logica

[clang] [mlir][OpenMP] Added `omp.structured_region` operation (PR #68825)

2023-10-12 Thread Kiran Chandramohan via cfe-commits
@@ -1987,4 +1987,40 @@ def ClauseRequiresAttr : EnumAttr { } + +def StructuredRegionOp : OpenMP_Op<"structured_region"> { + let summary = "Encapsulates a region in an OpenMP construct."; + let description = [{ +Encapsulates a region surrounded by an OpenMP Construct. T

[clang-tools-extra] [mlir][OpenMP] Added `omp.structured_region` operation (PR #68825)

2023-10-12 Thread Kiran Chandramohan via cfe-commits
@@ -1987,4 +1987,40 @@ def ClauseRequiresAttr : EnumAttr { } + +def StructuredRegionOp : OpenMP_Op<"structured_region"> { + let summary = "Encapsulates a region in an OpenMP construct."; + let description = [{ +Encapsulates a region surrounded by an OpenMP Construct. T

[clang-tools-extra] [mlir][OpenMP] Added `omp.structured_region` operation (PR #68825)

2023-10-12 Thread Kiran Chandramohan via cfe-commits
@@ -1468,6 +1468,18 @@ LogicalResult CancellationPointOp::verify() { return success(); } +//===--===// +// RegionOp +//===--===// + +Logica

[clang] f04ccad - [Flang] Add support for fsave-optimization-record

2023-07-28 Thread Kiran Chandramohan via cfe-commits
Author: Victor Kingi Date: 2023-07-28T09:26:40Z New Revision: f04ccadf35441b2e13b3ab30edeef251818fa9c7 URL: https://github.com/llvm/llvm-project/commit/f04ccadf35441b2e13b3ab30edeef251818fa9c7 DIFF: https://github.com/llvm/llvm-project/commit/f04ccadf35441b2e13b3ab30edeef251818fa9c7.diff LOG:

[clang] f48969f - Revert "[Flang][Sema] Move directive sets to a shared location"

2023-08-07 Thread Kiran Chandramohan via cfe-commits
Author: Kiran Chandramohan Date: 2023-08-07T16:43:42Z New Revision: f48969f90769f37e042025dba6c544eeddd6d3e6 URL: https://github.com/llvm/llvm-project/commit/f48969f90769f37e042025dba6c544eeddd6d3e6 DIFF: https://github.com/llvm/llvm-project/commit/f48969f90769f37e042025dba6c544eeddd6d3e6.diff

[clang] 6e13e3c - [Flang][Driver] Enable Rpass and other R family options.

2023-08-10 Thread Kiran Chandramohan via cfe-commits
Author: Victor Kingi Date: 2023-08-10T12:37:28+01:00 New Revision: 6e13e3c3e5e29ec0757370e354ed5e62147e47c7 URL: https://github.com/llvm/llvm-project/commit/6e13e3c3e5e29ec0757370e354ed5e62147e47c7 DIFF: https://github.com/llvm/llvm-project/commit/6e13e3c3e5e29ec0757370e354ed5e62147e47c7.diff

[clang] a969e05 - [OpenMP] Add Flush directive to OpenMPIRBuilder

2020-02-04 Thread Kiran Chandramohan via cfe-commits
Author: Kiran Chandramohan Date: 2020-02-04T22:48:02Z New Revision: a969e051a5d9fe17793e8be554b471429d97d958 URL: https://github.com/llvm/llvm-project/commit/a969e051a5d9fe17793e8be554b471429d97d958 DIFF: https://github.com/llvm/llvm-project/commit/a969e051a5d9fe17793e8be554b471429d97d958.diff

[clang] [flang] [clang][flang][driver] Correct program names in option group descriptions (PR #81726)

2024-02-14 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: Unrelated, but just wanted to bring this up since you are working in this area and distinguishing between Flang and Clang. Flang does not generate the Diagnostics Reference and hence ends up with the warning (see below) in the docs CI (https://lab.llvm.org/buildbot/#/b

[clang] [flang] [clang][flang][driver] Correct program names in option group descriptions (PR #81726)

2024-02-14 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. LGTM. Thanks @DavidSpickett for the fix. https://github.com/llvm/llvm-project/pull/81726 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/lis

[clang] [flang] [RFC][flang][runtime] Add FortranFloat128Math wrapper library. (PR #81971)

2024-02-20 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. Thanks Slava. https://github.com/llvm/llvm-project/pull/81971 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [flang][Driver] Support -Qunused-arguments (PR #107462)

2024-09-05 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. LG. https://github.com/llvm/llvm-project/pull/107462 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [mlir] [clang][flang][mlir] Support -frecord-command-line option (PR #102975)

2024-09-09 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. LGTM. https://github.com/llvm/llvm-project/pull/102975 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [mlir] [clang][flang][mlir] Support -frecord-command-line option (PR #102975)

2024-09-09 Thread Kiran Chandramohan via cfe-commits
@@ -1980,9 +1980,11 @@ within a command line are combined with spaces; spaces and backslashes within an argument are escaped with backslashes. This format differs from the format of the equivalent section produced by GCC with the -frecord-gcc-switches flag. This option is curr

[clang] [flang] [mlir] [clang][flang][mlir] Support -frecord-command-line option (PR #102975)

2024-08-13 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/102975 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [mlir] [clang][flang][mlir] Support -frecord-command-line option (PR #102975)

2024-08-13 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan commented: Couple of minor questions. https://github.com/llvm/llvm-project/pull/102975 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [mlir] [clang][flang][mlir] Support -frecord-command-line option (PR #102975)

2024-08-13 Thread Kiran Chandramohan via cfe-commits
@@ -1980,9 +1980,11 @@ within a command line are combined with spaces; spaces and backslashes within an argument are escaped with backslashes. This format differs from the format of the equivalent section produced by GCC with the -frecord-gcc-switches flag. This option is curr

[clang] [flang] [mlir] [clang][flang][mlir] Support -frecord-command-line option (PR #102975)

2024-08-13 Thread Kiran Chandramohan via cfe-commits
@@ -1980,9 +1980,11 @@ within a command line are combined with spaces; spaces and backslashes within an argument are escaped with backslashes. This format differs from the format of the equivalent section produced by GCC with the -frecord-gcc-switches flag. This option is curr

[clang] [flang] [Flang][Driver] Enable config file options (PR #100343)

2024-07-24 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan created https://github.com/llvm/llvm-project/pull/100343 Config files provide a facility to invoke the compiler with a predefined set of options. The patch only enables these options in the flang driver. Functionality was always there. >From bb8e77f755ee0c

[clang] [flang] [Flang][Driver] Enable config file options (PR #100343)

2024-07-24 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan milestoned https://github.com/llvm/llvm-project/pull/100343 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang][Driver] Enable config file options (PR #100343)

2024-07-24 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan closed https://github.com/llvm/llvm-project/pull/100343 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang][Driver] Enable config file options (PR #100343)

2024-07-25 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: /cherry-pick 8a77961 https://github.com/llvm/llvm-project/pull/100343 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-16 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: Could you add a flang driver test as well? https://github.com/llvm/llvm-project/pull/98736 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Add deprecation warning for `-Ofast` driver option (PR #98736)

2024-07-16 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: > > Could you add a flang driver test as well? > > The RFC was specifically about `-Ofast` in Clang and not behavior in Flang, > so what test behavior would you like to see? This patch currently deprecates > for both Clang and Flang, is that acceptable? I was asking f

[clang] [flang] [Flang][Driver] Add support for -w option (PR #90420)

2024-04-28 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan created https://github.com/llvm/llvm-project/pull/90420 A quick and dirty implementation of the -w option. Filters the warning messages generated by the Frontend during emission. TODO: Add more tests TODO: Ignore MLIR, LLVM IR and Driver warnings TODO: Chec

[clang] [flang] [Flang] Add support for -w option (PR #90420)

2024-04-28 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/90420 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang] Add support for -w option (PR #90420)

2024-04-28 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan updated https://github.com/llvm/llvm-project/pull/90420 >From 6a22c2de281bc04c9f71b7fcc8289e4217b70e08 Mon Sep 17 00:00:00 2001 From: Kiran Chandramohan Date: Fri, 26 Apr 2024 09:26:11 + Subject: [PATCH] [Flang][Driver] Add support for -w option A quick

[clang] [flang] [Flang] RFC: Add support for -w option (PR #90420)

2024-04-29 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/90420 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang] RFC: Add support for -w option (PR #90420)

2024-04-29 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/90420 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang] RFC: Add support for -w option (PR #90420)

2024-04-30 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: Thanks @jeffhammond. I will wait for @klausler to submit https://github.com/llvm/llvm-project/pull/90518 which will make it easier for the driver and will provide more fine-grained control of warnings. https://github.com/llvm/llvm-project/pull/90420 ___

[clang] [flang] [Flang] RFC: Add support for -w option (PR #90420)

2024-05-06 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan updated https://github.com/llvm/llvm-project/pull/90420 >From a250de8647af85e51e3eb7bf87c6a881a6de5ca0 Mon Sep 17 00:00:00 2001 From: Kiran Chandramohan Date: Fri, 26 Apr 2024 09:26:11 + Subject: [PATCH] [Flang][Driver] Add support for -w option 1/n Add

[clang] [flang] [Flang] RFC: Add support for -w option (PR #90420)

2024-05-06 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan updated https://github.com/llvm/llvm-project/pull/90420 >From a250de8647af85e51e3eb7bf87c6a881a6de5ca0 Mon Sep 17 00:00:00 2001 From: Kiran Chandramohan Date: Fri, 26 Apr 2024 09:26:11 + Subject: [PATCH] [Flang][Driver] Add support for -w option 1/n Add

[clang] [flang] [Flang] RFC: Add support for -w option (PR #90420)

2024-05-06 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan ready_for_review https://github.com/llvm/llvm-project/pull/90420 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang] RFC: Add support for -w option (PR #90420)

2024-05-06 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/90420 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang] RFC: Add support for -w option 1/n (PR #90420)

2024-05-06 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/90420 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang] RFC: Add support for -w option 1/n (PR #90420)

2024-05-07 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan updated https://github.com/llvm/llvm-project/pull/90420 >From 78e91d2d84638ded51267c5e0720cc1d1d4d773b Mon Sep 17 00:00:00 2001 From: Kiran Chandramohan Date: Fri, 26 Apr 2024 09:26:11 + Subject: [PATCH 1/2] [Flang][Driver] Add support for -w option 1/n

[clang] [flang] [Flang] RFC: Add support for -w option 1/n (PR #90420)

2024-05-07 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: Done. https://github.com/llvm/llvm-project/pull/90420 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang] RFC: Add support for -w option 1/n (PR #90420)

2024-05-07 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan updated https://github.com/llvm/llvm-project/pull/90420 >From 78e91d2d84638ded51267c5e0720cc1d1d4d773b Mon Sep 17 00:00:00 2001 From: Kiran Chandramohan Date: Fri, 26 Apr 2024 09:26:11 + Subject: [PATCH 1/2] [Flang][Driver] Add support for -w option 1/n

[clang] [flang] [Flang] RFC: Add support for -w option 1/n (PR #90420)

2024-05-08 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan closed https://github.com/llvm/llvm-project/pull/90420 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Driver] Restrict Ofast deprecation help message to Clang (PR #101682)

2024-08-02 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan created https://github.com/llvm/llvm-project/pull/101682 The discussion about this in Flang (https://discourse.llvm.org/t/rfc-deprecate-ofast-in-flang/80243) has not concluded hence restricting the deprecation only to Clang. >From e94fab5ceb88e5a6b44b6626

[clang] [flang] [Flang][Driver] Deprecate Ofast (PR #101701)

2024-08-02 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan created https://github.com/llvm/llvm-project/pull/101701 This is subject to agreement by the Flang community (https://discourse.llvm.org/t/rfc-deprecate-ofast-in-flang/80243). >From 60bd3cf870a078433cfa104bc872ad3083ff1e1c Mon Sep 17 00:00:00 2001 From: Kir

[clang] [Driver] Restrict Ofast deprecation help message to Clang (PR #101682)

2024-08-05 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan milestoned https://github.com/llvm/llvm-project/pull/101682 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Driver] Restrict Ofast deprecation help message to Clang (PR #101682)

2024-08-05 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan closed https://github.com/llvm/llvm-project/pull/101682 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Driver] Restrict Ofast deprecation help message to Clang (PR #101682)

2024-08-05 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: /cherry-pick e60ee1f https://github.com/llvm/llvm-project/pull/101682 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [clang][flang] Support -time in both clang and flang (PR #109165)

2024-10-15 Thread Kiran Chandramohan via cfe-commits
@@ -194,11 +197,29 @@ int Compilation::ExecuteCommand(const Command &C, if (LogOnly) return 0; + // We don't use any timers or llvm::TimeGroup's because those are tied into + // the global static timer list which, in principle, could be cleared without + // us knowing

[clang] [flang] [clang][flang] Support -time in both clang and flang (PR #109165)

2024-10-15 Thread Kiran Chandramohan via cfe-commits
@@ -194,11 +197,29 @@ int Compilation::ExecuteCommand(const Command &C, if (LogOnly) return 0; + // We don't use any timers or llvm::TimeGroup's because those are tied into + // the global static timer list which, in principle, could be cleared without + // us knowing

[clang] [flang] [clang][flang] Support -time in both clang and flang (PR #109165)

2024-10-15 Thread Kiran Chandramohan via cfe-commits
@@ -5865,6 +5865,7 @@ def print_enabled_extensions : Flag<["-", "--"], "print-enabled-extensions">, def : Flag<["-"], "mcpu=help">, Alias; def : Flag<["-"], "mtune=help">, Alias; def time : Flag<["-"], "time">, + Visibility<[ClangOption, CLOption, DXCOption, FlangOption]>, --

[clang] [flang] [clang][flang] Support -time in both clang and flang (PR #109165)

2024-10-15 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. Thanks Tarun. LGTM. https://github.com/llvm/llvm-project/pull/109165 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [llvm] [openmp] [flang][driver] rename flang-new to flang (PR #110023)

2024-10-08 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. LGTM. Thanks for adding the warning about OpenMP. Please ask in the Flang community call tomorrow and in the discourse post before you submit. https://github.com/llvm/llvm-project/pull/110023 _

[clang] [clang] Make -fveclib={ArmPL, SLEEF} imply -fno-math-errno (PR #112580)

2024-10-16 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan commented: Thanks for this patch. Have you checked the flang driver? Is it not applicable there since errno is not used in Flang? https://github.com/llvm/llvm-project/pull/112580 ___ cfe-commits mailing list cfe-c

[clang] [clang] Make -fveclib={ArmPL, SLEEF} imply -fno-math-errno (PR #112580)

2024-10-16 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/112580 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Make -fveclib={ArmPL, SLEEF} imply -fno-math-errno (PR #112580)

2024-10-16 Thread Kiran Chandramohan via cfe-commits
@@ -3125,6 +3129,10 @@ static void RenderFloatingPointOptions(const ToolChain &TC, const Driver &D, TrappingMathPresent = true; FPExceptionBehavior = "strict"; break; +case options::OPT_fveclib: + if (llvm::is_contained(VecLibImpliesNoMathErrno, A->g

[clang] [flang] [llvm] [openmp] [flang][driver] rename flang-new to flang (PR #110023)

2024-10-10 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: The 2 stage buildbot (https://lab.llvm.org/buildbot/#/builders/41) with the Fortran tests passed after this merge. Its latest run is also a pass. https://github.com/llvm/llvm-project/pull/110023 ___ cfe-commits mailing list cf

[clang] [flang] [llvm] [openmp] [flang][driver] rename flang-new to flang (PR #110023)

2024-10-10 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: @everythingfunctional Following are the steps to run the testsuite. Please check and confirm if you did not run it before. ``` git clone https://github.com/llvm/llvm-test-suite.git cd llvm-test-suite mkdir build cd build cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_C_COMPI

[clang] [flang] [llvm] [openmp] [flang][driver] rename flang-new to flang (PR #110023)

2024-10-10 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan closed https://github.com/llvm/llvm-project/pull/110023 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Make -fveclib={ArmPL, SLEEF} imply -fno-math-errno (PR #112580)

2024-10-23 Thread Kiran Chandramohan via cfe-commits
@@ -3410,7 +3410,8 @@ def fno_experimental_isel : Flag<["-"], "fno-experimental-isel">, Group; def fveclib : Joined<["-"], "fveclib=">, Group, Visibility<[ClangOption, CC1Option, FlangOption, FC1Option]>, -HelpText<"Use the given vector functions library">, +HelpText<

[clang] [flang] [llvm] Enable seq_cst Clause for flush Directive in OpenMP (PR #114072)

2024-11-04 Thread Kiran Chandramohan via cfe-commits
@@ -17,6 +17,7 @@ template T tmain(T argc) { static T a; #pragma omp flush +#pragma omp flush seq_cst kiranchandramohan wrote: We need a CHECK line corresponding to this new entry. https://github.com/llvm/llvm-project/pull/114072 __

[clang] [flang] [llvm] Enable seq_cst Clause for flush Directive in OpenMP (PR #114072)

2024-11-04 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/114072 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [llvm] Enable seq_cst Clause for flush Directive in OpenMP (PR #114072)

2024-11-04 Thread Kiran Chandramohan via cfe-commits
@@ -28,6 +29,7 @@ T tmain(T argc) { int main() { static int a; #pragma omp flush +#pragma omp flush seq_cst kiranchandramohan wrote: We need a CHECK line corresponding to this new entry. https://github.com/llvm/llvm-project/pull/114072 _

[clang] [flang] [llvm] Enable seq_cst Clause for flush Directive in OpenMP (PR #114072)

2024-11-04 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan commented: Thanks for this patch. Could you update the title/summary of the patch to say that seq_cst is allowed in Flush since OpenMP 5.1. Also, might make sense to say why no other changes in Lowering to IR is required. Could you add a test in flang/test/

[clang] [clang] Make -fveclib={ArmPL, SLEEF} imply -fno-math-errno (PR #112580)

2024-10-23 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. LG. https://github.com/llvm/llvm-project/pull/112580 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [llvm] seq_cst is allowed in Flush since OpenMP 5.1. (PR #114072)

2024-11-13 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: > > Thanks for this patch. Could you update the title/summary of the patch to > > say that seq_cst is allowed in Flush since OpenMP 5.1. Also, might make > > sense to say why no other changes in Lowering to IR is required. > > Could you add a test in flang/test/Lower/Op

[clang] [llvm] [Transforms][Utils][PromoteMem2Reg] Propagate nnan flag on par with the nsz flag (PR #114271)

2024-11-01 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/114271 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang][Driver] Enable the -B option (PR #109965)

2024-09-26 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan updated https://github.com/llvm/llvm-project/pull/109965 >From 3af5907719193ebc5b251b4681c137827550a1d0 Mon Sep 17 00:00:00 2001 From: Kiran Chandramohan Date: Wed, 25 Sep 2024 11:43:40 + Subject: [PATCH 1/2] [Flang][Driver] Enable the -B option The opt

[clang] [flang] [Flang][Driver] Enable the -B option (PR #109965)

2024-09-26 Thread Kiran Chandramohan via cfe-commits
@@ -0,0 +1,23 @@ +! Check -B driver option. +! +! Target triple prefix is not detected for -B. +! RUN: %flang %s -### -o %t.o -target i386-unknown-linux \ kiranchandramohan wrote: Thanks. Done. Modified tests for both drivers. https://github.com/llvm/llvm-projec

[clang] [flang] [flang][Driver] Support -fdiagnostics-color (PR #109210)

2024-09-26 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. LG. Thanks Tarun. https://github.com/llvm/llvm-project/pull/109210 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [mlir] [clang][flang][mlir] Reapply "Support -frecord-command-line option (#102975)" (PR #110132)

2024-09-26 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. LG. https://github.com/llvm/llvm-project/pull/110132 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang][Driver][Offload] Support -Xoffload-linker argument in Flang (PR #109907)

2024-09-27 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. LG. https://github.com/llvm/llvm-project/pull/109907 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [Flang][Driver] Enable the -B option (PR #109965)

2024-09-30 Thread Kiran Chandramohan via cfe-commits
@@ -0,0 +1,23 @@ +! Check -B driver option. +! +! Target triple prefix is not detected for -B. +! RUN: %flang %s -### -o %t.o -target i386-unknown-linux \ kiranchandramohan wrote: @MaskRay Is it OK to land this? https://github.com/llvm/llvm-project/pull/109965 _

[clang] [flang] [Flang][Driver] Enable the -B option (PR #109965)

2024-10-01 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan closed https://github.com/llvm/llvm-project/pull/109965 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [flang][Driver] Support -fdiagnostics-color (PR #109210)

2024-09-19 Thread Kiran Chandramohan via cfe-commits
@@ -732,6 +732,12 @@ void Flang::ConstructJob(Compilation &C, const JobAction &JA, // to avoid warn_drv_unused_argument. Args.getLastArg(options::OPT_fcolor_diagnostics, options::OPT_fno_color_diagnostics); + if (const Arg *A = Args.getLastArg(options::O

[clang] [flang] [Flang][Driver] Enable the -B option (PR #109965)

2024-09-25 Thread Kiran Chandramohan via cfe-commits
@@ -0,0 +1,23 @@ +! Check -B driver option. +! +! Target triple prefix is not detected for -B. +! RUN: %flang %s -### -o %t.o -target i386-unknown-linux \ +! RUN: -B %S/Inputs/B_opt_tree/dir1 -fuse-ld=ld 2>&1 \ +! RUN: | FileCheck --check-prefix=CHECK-B-OPT-TRIPLE %s +! CHEC

[clang] [flang] [Flang][Driver] Enable the -B option (PR #109965)

2024-09-25 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan created https://github.com/llvm/llvm-project/pull/109965 The option provides the search prefix for executables, libraries and data files. The option is implemented in the common portion of the Driver and only needs to be enabled in Flang. Test added is a co

[clang] [flang] [Flang][Driver] Enable the -B option (PR #109965)

2024-09-25 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan updated https://github.com/llvm/llvm-project/pull/109965 >From 322ff8178ae23b43168d88fe8ab314e5ac3ffb72 Mon Sep 17 00:00:00 2001 From: Kiran Chandramohan Date: Wed, 25 Sep 2024 11:43:40 + Subject: [PATCH] [Flang][Driver] Enable the -B option The option

[clang] [flang] [Flang][Driver] Enable the -B option (PR #109965)

2024-09-25 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan updated https://github.com/llvm/llvm-project/pull/109965 >From 3af5907719193ebc5b251b4681c137827550a1d0 Mon Sep 17 00:00:00 2001 From: Kiran Chandramohan Date: Wed, 25 Sep 2024 11:43:40 + Subject: [PATCH] [Flang][Driver] Enable the -B option The option

[clang] [flang] [clang][driver] When -fveclib=ArmPL flag is in use, always link against libamath (PR #116432)

2024-11-18 Thread Kiran Chandramohan via cfe-commits
kiranchandramohan wrote: We probably have to provide a link to some documentation for libamath. May be something like the following one. https://developer.arm.com/documentation/102574/2410/Optimized-math-routines---libamath https://github.com/llvm/llvm-project/pull/116432 _

[clang] [flang] [flang][driver] add option to make all main program variable static (PR #121968)

2025-01-07 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/121968 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [flang][driver] add option to make all main program variable static (PR #121968)

2025-01-07 Thread Kiran Chandramohan via cfe-commits
@@ -6945,6 +6945,9 @@ defm unsigned : OptInFC1FFlag<"unsigned", "Enables UNSIGNED type">; def fno_automatic : Flag<["-"], "fno-automatic">, Group, HelpText<"Implies the SAVE attribute for non-automatic local objects in subprograms unless RECURSIVE">; +def fsave_main_progra

[clang] [flang] [flang][driver] add option to make all main program variable static (PR #121968)

2025-01-07 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan approved this pull request. LG. Thanks @jeanPerier. https://github.com/llvm/llvm-project/pull/121968 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

[clang] [flang] [flang][OpenMP] Upstream first part of `do concurrent` mapping (PR #126026)

2025-02-12 Thread Kiran Chandramohan via cfe-commits
https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/126026 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [flang][OpenMP] Upstream first part of `do concurrent` mapping (PR #126026)

2025-02-12 Thread Kiran Chandramohan via cfe-commits
@@ -0,0 +1,380 @@ + + +# `DO CONCURENT` mapping to OpenMP + +```{contents} +--- +local: +--- +``` + +This document seeks to describe the effort to parallelize `do concurrent` loops +by mapping them to OpenMP worksharing constructs. The goals of this document +are: +* Describing ho

[clang] [flang] [flang][OpenMP] Upstream first part of `do concurrent` mapping (PR #126026)

2025-02-12 Thread Kiran Chandramohan via cfe-commits
@@ -6919,6 +6919,10 @@ defm loop_versioning : BoolOptionWithoutMarshalling<"f", "version-loops-for-stri def fhermetic_module_files : Flag<["-"], "fhermetic-module-files">, Group, HelpText<"Emit hermetic module files (no nested USE association)">; + +def do_concurrent_to_op

<    1   2   3   4   >