https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/66702
>From f149c390f892c4e3f4b67ac9ae833b487b7c026d Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 18 Sep 2023 15:18:24 -0600
Subject: [PATCH 1/7] [flang][Driver] Support -rpath, -shared, and -static in
t
@@ -48,3 +48,17 @@
! MSVC-SAME: FortranDecimal.lib
! MSVC-SAME: /subsystem:console
! MSVC-SAME: "[[object_file]]"
+
+! Verify that certain linker flags are known to the frontend and are passed on
+! to the linker.
+
+! RUN: %flang -### --target=x86_64-linux-gnu -rpath /path/to/
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/66702
>From f149c390f892c4e3f4b67ac9ae833b487b7c026d Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 18 Sep 2023 15:18:24 -0600
Subject: [PATCH 1/8] [flang][Driver] Support -rpath, -shared, and -static in
t
tarunprabhu wrote:
clang has a
[test](https://github.com/llvm/llvm-project/blob/main/clang/test/Driver/dynamic-linker.c)
that is similar and in a file with that name.
https://github.com/llvm/llvm-project/pull/66702
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/66702
>From 68733f1502ba23c4c6bb2f172bd9bae8362317d3 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 18 Sep 2023 15:18:24 -0600
Subject: [PATCH] [flang][Driver] Support -rpath, -shared, and -static in the
f
https://github.com/tarunprabhu closed
https://github.com/llvm/llvm-project/pull/66702
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tarunprabhu created
https://github.com/llvm/llvm-project/pull/77360
The -pthread option is supported by both clang and gfortran.
The -pthread option seems to be extensively tested for various platforms by
clang. We should be able to piggy-back on those tests since we use the
tarunprabhu wrote:
Thanks, I am not sure how I missed that.
@banach-space, Is the intention for the `-pthread` option to be illegal in
flang?
At the last flang community call
([2024-01-03](https://docs.google.com/document/d/1Z2U5UAtJ-Dag5wlMaLaW1KRmNgENNAYynJqLW2j2AZQ/edit#heading=h.de8wft8m
tarunprabhu wrote:
@banach-space.
Thanks for clarifying. I'll look into creating a test that fails without
`-pthread`.
https://github.com/llvm/llvm-project/pull/77360
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi
tarunprabhu wrote:
> IIUC, this means that on older system compilation will indeed fail without
> `-pthread`, but shouldn't be needed on newer systems. @tarunprabhu - perhaps
> add a link to that article in your test and add a note that on many systems
> compilation will succeed even without t
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/77360
>From 187f91dcd9e602c944b089adfb243127c31de2ca Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 8 Jan 2024 10:56:09 -0700
Subject: [PATCH 1/2] [flang][Driver] Support -pthread in the frontend
The -pthr
tarunprabhu wrote:
> [nit] Note that the test that you've added will run on any platform. Unless
> that's referring to something else?
That was only highlighting that I have not tested this on AArch64 or Windows.
Could someone test this on AArch64 and/or Windows just to make sure that it
pas
tarunprabhu wrote:
> A few of the tests in the gfortran-testsuite using polymorphism fail at
> runtime. I'm happy for this PR to be merged as it is and these bugs to be
> fixed later.
Once this is merged in, the polymorphism tests can be enabled together with the
larger group of tests that I
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/77360
>From ab27280e85d6027e6ff119c0803d962753e83b52 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 8 Jan 2024 10:56:09 -0700
Subject: [PATCH] [flang][Driver] Support -pthread in the frontend
The -pthread
https://github.com/tarunprabhu closed
https://github.com/llvm/llvm-project/pull/77360
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/66702
>From 3302fb4937d75c0920e7bcf8f215e12fbe770a36 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 18 Sep 2023 15:18:24 -0600
Subject: [PATCH 1/6] [flang][Driver] Support -rpath, -shared, and -static in
t
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/66702
>From ba3bb1ae28c14f6720e17d26e037c7406a638436 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 18 Sep 2023 15:18:24 -0600
Subject: [PATCH 1/6] [flang][Driver] Support -rpath, -shared, and -static in
t
Author: Tarun Prabhu
Date: 2022-10-04T17:02:45-06:00
New Revision: 43fe6f7cc35ded691bbc2fa844086d321e705d46
URL:
https://github.com/llvm/llvm-project/commit/43fe6f7cc35ded691bbc2fa844086d321e705d46
DIFF:
https://github.com/llvm/llvm-project/commit/43fe6f7cc35ded691bbc2fa844086d321e705d46.diff
Author: Tarun Prabhu
Date: 2022-11-10T08:03:46-07:00
New Revision: c3821b8d2aacd1d7c0281db1b8db011e1158cf4d
URL:
https://github.com/llvm/llvm-project/commit/c3821b8d2aacd1d7c0281db1b8db011e1158cf4d
DIFF:
https://github.com/llvm/llvm-project/commit/c3821b8d2aacd1d7c0281db1b8db011e1158cf4d.diff
https://github.com/tarunprabhu created
https://github.com/llvm/llvm-project/pull/66702
Enable -rpath, -shared, and -static for the flang frontend. This brings it in
line with clang. Fixes issue #65546.
>From 3302fb4937d75c0920e7bcf8f215e12fbe770a36 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
@@ -50,3 +50,19 @@
! MSVC-SAME: FortranDecimal.lib
! MSVC-SAME: /subsystem:console
! MSVC-SAME: "[[object_file]]"
+
+! Verify that certain linker flags are known to the frontend and are passed on
+! to the linker.
+
+! RUN: %flang -### -rpath /path/to/dir %s 2>&1 \
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/66702
>From 3302fb4937d75c0920e7bcf8f215e12fbe770a36 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 18 Sep 2023 15:18:24 -0600
Subject: [PATCH 1/2] [flang][Driver] Support -rpath, -shared, and -static in
t
@@ -50,3 +50,19 @@
! MSVC-SAME: FortranDecimal.lib
! MSVC-SAME: /subsystem:console
! MSVC-SAME: "[[object_file]]"
+
+! Verify that certain linker flags are known to the frontend and are passed on
+! to the linker.
+
+! RUN: %flang -### -rpath /path/to/dir %s 2>&1 \
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/66702
>From 3302fb4937d75c0920e7bcf8f215e12fbe770a36 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 18 Sep 2023 15:18:24 -0600
Subject: [PATCH 1/3] [flang][Driver] Support -rpath, -shared, and -static in
t
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/66702
>From 3302fb4937d75c0920e7bcf8f215e12fbe770a36 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 18 Sep 2023 15:18:24 -0600
Subject: [PATCH 1/4] [flang][Driver] Support -rpath, -shared, and -static in
t
tarunprabhu wrote:
Thanks @MaskRay and @banach-space for taking a look at this. I have marked the
test as unsupported on windows because of a failing buildkite. I don't have
access to a Windows machine so I can't add an equivalent test for that platform.
https://github.com/llvm/llvm-project/pu
https://github.com/tarunprabhu resolved
https://github.com/llvm/llvm-project/pull/66702
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tarunprabhu resolved
https://github.com/llvm/llvm-project/pull/66702
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tarunprabhu wrote:
> I think Windows does not work likely because `ld` is absent in PATH. You need
> `-Bxxx/bin` to specify a directory that contains `ld`, but I don't remember
> whether `ld.exe` is needed instead.
I think what I saw on the buildkite was Link.exe, but I don't know if that is
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/66702
>From 3302fb4937d75c0920e7bcf8f215e12fbe770a36 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 18 Sep 2023 15:18:24 -0600
Subject: [PATCH 1/5] [flang][Driver] Support -rpath, -shared, and -static in
t
tarunprabhu wrote:
The buildkite
[fails](https://buildkite.com/llvm-project/github-pull-requests/builds/3888#018acca9-e640-47a6-bfe8-3831aeddeb41)
yet again because the link flags seem to be mapped differently.
The link line generated on Windows is:
`"C:\\BuildTools\\VC\\Tools\\MSVC\\14.29.3
tarunprabhu wrote:
Thanks @DavidTruby.
For this particular patch, the main concern is how the compiler flags are
mapped to the corresponding linker flags on Windows. All of that work is done
by clang, and I haven't touched any of it so I am not worried about
correctness.
The two main conce
tarunprabhu wrote:
> I'll try to test this patch on a Windows on ARM machine. But it may take some
> time to setup and build flang on it.
Thanks @luporl. Any help is most appreciated.
https://github.com/llvm/llvm-project/pull/66702
___
cfe-commits ma
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/107270
>From 2f4a8503c605f7401bf9312b59a72b6b3ccdbb7f Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Wed, 4 Sep 2024 10:24:31 -0600
Subject: [PATCH 1/2] [flang][Driver] Preliminary support for -ftime-report
The
@@ -143,6 +144,14 @@ class CompilerInvocation : public CompilerInvocationBase {
},
};
+ /// Whether to time the invocation. Set when -ftime-report or -ftime-report=
+ /// is enabled.
+ bool enableTimers;
+
+ /// Whether to report the timing of each run of an LLVM p
@@ -4,6 +4,7 @@ add_subdirectory(Decimal)
add_subdirectory(Lower)
add_subdirectory(Parser)
add_subdirectory(Semantics)
+add_subdirectory(Support)
tarunprabhu wrote:
Mainly to match the layout in both `llvm` and `mlir`. In principle, it could be
used in other
@@ -254,6 +276,43 @@ class CompilerInstance {
/// Produces the string which represents target feature
std::string getTargetFeatures();
+ /// {
+ /// @name Timing
+ /// @{
+ bool isTimingEnabled() { return timingMgr.isEnabled(); }
tarunprabhu wrote:
Go
@@ -176,6 +205,26 @@ bool CompilerInstance::executeAction(FrontendAction &act) {
act.endSourceFile();
}
}
+
+ if (timingMgr.isEnabled()) {
+timingScopeRoot.stop();
+
+// Write the timings to the associated output stream and clear all timers.
+// We need
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/107270
>From 2f4a8503c605f7401bf9312b59a72b6b3ccdbb7f Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Wed, 4 Sep 2024 10:24:31 -0600
Subject: [PATCH 1/3] [flang][Driver] Preliminary support for -ftime-report
The
@@ -143,6 +144,14 @@ class CompilerInvocation : public CompilerInvocationBase {
},
};
+ /// Whether to time the invocation. Set when -ftime-report or -ftime-report=
+ /// is enabled.
+ bool enableTimers;
+
+ /// Whether to report the timing of each run of an LLVM p
https://github.com/tarunprabhu created
https://github.com/llvm/llvm-project/pull/107455
Because of the way visibility is implemented in Options.td, options that are
aliases do not inherit the visibility of the option being aliased. Therefore,
explicitly set the visibility of the alias to be th
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/107455
>From 984032c16fd7ebadf06737973ad3a165b689cc3c Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Thu, 5 Sep 2024 13:34:51 -0600
Subject: [PATCH] [flang][Driver] Support --no-warnings option
Because of the w
https://github.com/tarunprabhu created
https://github.com/llvm/llvm-project/pull/107462
This partially addresses: https://github.com/llvm/llvm-project/issues/89888
>From 302ee0694f6e2528076a4e2a98ca37213e78efae Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Thu, 5 Sep 2024 14:33:50 -0600
Su
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/102975
>From 6df2f16e9b875f598cb00738b6fb9e8a2d7fe448 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH] [clang][flang][mlir] Support -frecord-command-line option
Ad
https://github.com/tarunprabhu edited
https://github.com/llvm/llvm-project/pull/107472
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tarunprabhu closed
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
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/107472
>From acb5cf2af69f7941c24c4e53b67f9f2d4a18dbc8 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Thu, 5 Sep 2024 15:41:54 -0600
Subject: [PATCH 1/2] [flang][Driver] Support -Xlinker in flang
---
clang/incl
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/107472
>From acb5cf2af69f7941c24c4e53b67f9f2d4a18dbc8 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Thu, 5 Sep 2024 15:41:54 -0600
Subject: [PATCH 1/3] [flang][Driver] Support -Xlinker in flang
---
clang/incl
@@ -0,0 +1,6 @@
+! RUN: %flang -### -o /dev/null %s -Xlinker -rpath=/not/a/real/path 2>&1 |
FileCheck %s
+
+! CHECK: "-fc1"
+! CHECK-NEXT: "-rpath=/not/a/real/path"
tarunprabhu wrote:
Thanks. I have updated the tests quite extensively. There are now checks for
tarunprabhu wrote:
> Thanks for the updates. LGTM
Thanks for the reviews, Tom :-)
https://github.com/llvm/llvm-project/pull/107472
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,8 @@
+! RUN: %flang -### -o /dev/null %s -Xlinker -rpath=/not/a/real/path 2>&1 |
FileCheck --check-prefix=SINGLE %s
+! RUN: %flang -### -o /dev/null %s -Xlinker -rpath -Xlinker /not/a/real/path
2>&1 | FileCheck --check-prefix=MULTIPLE %s
+
+
+! SINGLE: "-rpath=/not/a/
https://github.com/tarunprabhu closed
https://github.com/llvm/llvm-project/pull/107455
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/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
https://github.com/tarunprabhu closed
https://github.com/llvm/llvm-project/pull/107472
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/102975
>From d5f48306117d0e3a166837b48ba475d70e15c6c1 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH 1/2] [clang][flang][mlir] Support -frecord-command-line option
https://github.com/tarunprabhu created
https://github.com/llvm/llvm-project/pull/102975
Add support for the -frecord-command-line option that will produce the
llvm.commandline metadata which will eventually be saved in the object file.
This behavior is also supported in clang. Some refactoring
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/102975
>From 4ad7fc6a206a1d4004fe11013bce9ebee0347479 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH 1/2] [clang][flang][mlir] Support -frecord-command-line option
https://github.com/tarunprabhu 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
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/102975
>From 4ad7fc6a206a1d4004fe11013bce9ebee0347479 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH 1/3] [clang][flang][mlir] Support -frecord-command-line option
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/102975
>From 4ad7fc6a206a1d4004fe11013bce9ebee0347479 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH 1/4] [clang][flang][mlir] Support -frecord-command-line option
@@ -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
@@ -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
https://github.com/tarunprabhu commented:
Is there a way to test that the attribute is lowered to FIR and perhaps also on
to LLVM? `flang/test/Lower/` has some tests which check for the presence of
certain constructs in the LLVM IR. Could something be written similar to those?
https://github.c
@@ -371,7 +371,7 @@ static mlir::LogicalResult convertFortranSourceToMLIR(
ctx, semanticsContext, defKinds, semanticsContext.intrinsics(),
semanticsContext.targetCharacteristics(), parsing.allCooked(),
targetTriple, kindMap, loweringOptions, envDefaults,
-
https://github.com/tarunprabhu edited
https://github.com/llvm/llvm-project/pull/95043
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -411,6 +412,13 @@ void Flang::addTargetOptions(const ArgList &Args,
}
// TODO: Add target specific flags, ABI, mtune option etc.
+ if (const Arg *A = Args.getLastArg(options::OPT_mtune_EQ)) {
+CmdArgs.push_back("-tune-cpu");
+if (strcmp(A->getValue(), "native")
tarunprabhu wrote:
Why do you have the first `target-cpu` check on its own? Is it intended to
check for the _absence_ of the `tune-cpu` attribute when `target-cpu` is
provided on the command line? If so, there should be a `CPU-NOT` annotation as
well. Otherwi
tarunprabhu wrote:
Consider replacing `#0` with `#{{[0-9]+}}`. While the attributes are almost
certainly going to be in `#0` for such a small function, you don't actually
_require_ this to match `#0`, so you it might be better to let it match any
number.
Al
https://github.com/tarunprabhu edited
https://github.com/llvm/llvm-project/pull/95043
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -371,7 +371,7 @@ static mlir::LogicalResult convertFortranSourceToMLIR(
ctx, semanticsContext, defKinds, semanticsContext.intrinsics(),
semanticsContext.targetCharacteristics(), parsing.allCooked(),
targetTriple, kindMap, loweringOptions, envDefaults,
-
https://github.com/tarunprabhu commented:
Thanks for the changes. LGTM, but wait for @banach-space to approve.
https://github.com/llvm/llvm-project/pull/95043
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/106141
>From 9b83df995dc5c1d95db63a6ad32ba612b8a52290 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 26 Aug 2024 14:51:29 -0600
Subject: [PATCH 1/3] [flang][Driver] Add support for -mllvm
-print-pipeline-p
tarunprabhu wrote:
>
> To unblock you, I propose the following:
>
> 1. Keep this as is, but add a comment saying that this should be a proper
> compiler option instead of a secret handshake like this. Similar comment
> should be added in Clang. I will happily approve that.
>
I have adde
https://github.com/tarunprabhu created
https://github.com/llvm/llvm-project/pull/96678
Reverts llvm/llvm-project#95043
>From b07dae1f1b758e9f55667dac4db38c78bd609656 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Tue, 25 Jun 2024 12:56:50 -0600
Subject: [PATCH] Revert "[flang] Add basic -mt
https://github.com/tarunprabhu closed
https://github.com/llvm/llvm-project/pull/96678
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tarunprabhu wrote:
> Why do we need the two new files?
>
> We have clang/lib/Driver/ToolChains/CommonArgs.cpp and
> clang/lib/Driver/ToolChain.cpp for shared code.
Since most of the code in `CommonArgs.cpp` had to do with functions in the
`clang::driver::tools` namespace, I wasn't sure that c
https://github.com/tarunprabhu created
https://github.com/llvm/llvm-project/pull/108868
This partially addresses some requests in #89888
>From 612b93eda5695306293bb8ca6e27a810b06adf3d Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 16 Sep 2024 12:08:38 -0600
Subject: [PATCH] [flang][Dri
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/102975
>From 4a37cec543f30bb122bf14573fdec8302a0afa3e Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH 1/3] [clang][flang][mlir] Support -frecord-command-line option
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/102975
>From 4a37cec543f30bb122bf14573fdec8302a0afa3e Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH 1/4] [clang][flang][mlir] Support -frecord-command-line option
@@ -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
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/109165
>From 3b926fce8dbf42577adf54e80d3383a85b170e91 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Wed, 18 Sep 2024 09:49:26 -0600
Subject: [PATCH 1/6] [clang][flang] Support -time in both clang and flang
The
@@ -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
@@ -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
@@ -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]>,
--
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/109165
>From f0540da79936c008a8fea8a16cefda6398d59ed4 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Wed, 18 Sep 2024 09:49:26 -0600
Subject: [PATCH 1/7] [clang][flang] Support -time in both clang and flang
The
@@ -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
@@ -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
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/109165
>From 6ff1fcb6dcd55b1d77dacc8b6d117176033038c0 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Wed, 18 Sep 2024 09:49:26 -0600
Subject: [PATCH 1/5] [clang][flang] Support -time in both clang and flang
The
@@ -0,0 +1,24 @@
+// The -time option prints timing information for the various subcommands in a
+// format similar to that used by gcc. When compiling and linking, this will
+// include the time to call clang-${LLVM_VERSION_MAJOR} and the linker. Since
+// the name of the linker
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/110132
>From 8564770eaa5636abe9a3624619937b0235df0e37 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH] [clang][flang][mlir] Support -frecord-command-line option
Ad
https://github.com/tarunprabhu approved this pull request.
Thank you for the changes, Minato san. LGTM.
https://github.com/llvm/llvm-project/pull/110061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
@@ -0,0 +1,24 @@
+// The -time option prints timing information for the various subcommands in a
+// format similar to that used by gcc. When compiling and linking, this will
+// include the time to call clang-${LLVM_VERSION_MAJOR} and the linker. Since
+// the name of the linker
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/110132
>From e307d2d0d0407c4e6f910a77cc5ae5cd97d08647 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH] [clang][flang][mlir] Support -frecord-command-line option
Ad
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/110132
>From 797927dc0858886fa5ecced02903f98bdc929121 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH] [clang][flang][mlir] Support -frecord-command-line option
Ad
@@ -866,6 +866,17 @@ void Flang::ConstructJob(Compilation &C, const JobAction
&JA,
}
}
+ // -fno-strict-overflow implies -fwrapv if it isn't disabled, but
+ // -fstrict-overflow won't turn off an explicitly enabled -fwrapv.
+ if (Arg *A = Args.getLastArg(options::OPT
https://github.com/tarunprabhu closed
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
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/110132
>From 61b13cf54de802351e785f50df77adc30d5e7695 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 12 Aug 2024 14:32:08 -0600
Subject: [PATCH] [clang][flang][mlir] Support -frecord-command-line option
Ad
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/109165
>From 3b926fce8dbf42577adf54e80d3383a85b170e91 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Wed, 18 Sep 2024 09:49:26 -0600
Subject: [PATCH 1/5] [clang][flang] Support -time in both clang and flang
The
tarunprabhu wrote:
@MaskRay, have all your comments been addressed?
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
https://github.com/tarunprabhu closed
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
1 - 100 of 237 matches
Mail list logo
