https://github.com/tblah closed https://github.com/llvm/llvm-project/pull/133745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -6961,6 +6960,52 @@ 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">;
+defm repack_arrays
+
https://github.com/tblah approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/134002
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1687,6 +1700,7 @@ void CompilerInvocation::setLoweringOptions() {
const Fortran::common::LangOptions &langOptions = getLangOpts();
loweringOpts.setIntegerWrapAround(langOptions.getSignedOverflowBehavior() ==
Fortran::common::LangOptio
https://github.com/tblah edited https://github.com/llvm/llvm-project/pull/134002
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah created
https://github.com/llvm/llvm-project/pull/133745
Precise OpenMP standards support information is being documented in #132707
Flang now has good support for OpenMP Version 3.1 and earlier.
>From 3bce8b379641d8e91c8e8ae389a113f8d882b14e Mon Sep 17 00:00:00 2001
F
https://github.com/tblah closed https://github.com/llvm/llvm-project/pull/132801
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1,10 +1,15 @@
! RUN: %flang -### -S -fslp-vectorize %s 2>&1 | FileCheck
-check-prefix=CHECK-SLP-VECTORIZE %s
! RUN: %flang -### -S -fno-slp-vectorize %s 2>&1 | FileCheck
-check-prefix=CHECK-NO-SLP-VECTORIZE %s
+! RUN: %flang -### -S -O0 %s 2>&1 | FileCheck
-check-prefix=C
@@ -0,0 +1,10 @@
+! RUN: %flang -### -S -fslp-vectorize %s 2>&1 | FileCheck
-check-prefix=CHECK-SLP-VECTORIZE %s
+! RUN: %flang -### -S -fno-slp-vectorize %s 2>&1 | FileCheck
-check-prefix=CHECK-NO-SLP-VECTORIZE %s
+! RUN: %flang -### -S -O1 %s 2>&1 | FileCheck
-check-prefix=CH
@@ -0,0 +1,826 @@
+//===- BuildBuiltins.cpp - Utility builder for builtins
---===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/tblah commented:
The idea behind this sounds good to me, and this patch improves flang's test
results on the fujitsu testsutie, shows no regressions in the gfortran test
suite, and fixes the important remaining flang issues from omp atomic. Atomic
support is one of the most
https://github.com/tblah edited https://github.com/llvm/llvm-project/pull/101966
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah closed https://github.com/llvm/llvm-project/pull/130788
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah updated
https://github.com/llvm/llvm-project/pull/130788
>From 5d3c1cc1d40b308b93cc47c78c6c3ca677f55155 Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Tue, 11 Mar 2025 15:54:35 +
Subject: [PATCH 1/4] [flang] Add support for -f[no-]verbose-asm
This flag provides ex
https://github.com/tblah edited https://github.com/llvm/llvm-project/pull/130788
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tblah wrote:
No problem. I have moved it to target options in
https://github.com/llvm/llvm-project/pull/130788/commits/6f6f8670887c9eb030412a7eb720242bba2790db
https://github.com/llvm/llvm-project/pull/130788
___
cfe-commits mailing list
cfe-commits@l
https://github.com/tblah updated
https://github.com/llvm/llvm-project/pull/130788
>From 5d3c1cc1d40b308b93cc47c78c6c3ca677f55155 Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Tue, 11 Mar 2025 15:54:35 +
Subject: [PATCH 1/3] [flang] Add support for -f[no-]verbose-asm
This flag provides ex
https://github.com/tblah updated
https://github.com/llvm/llvm-project/pull/129872
>From 3055b89b8ab937bfd4ba70d87e1733b05156d0ac Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Tue, 4 Mar 2025 17:19:53 +
Subject: [PATCH 1/3] [mlir][OpenMP] fix crash outlining infinite loop
Previously an ex
https://github.com/tblah updated
https://github.com/llvm/llvm-project/pull/130788
>From 5d3c1cc1d40b308b93cc47c78c6c3ca677f55155 Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Tue, 11 Mar 2025 15:54:35 +
Subject: [PATCH 1/2] [flang] Add support for -f[no-]verbose-asm
This flag provides ex
https://github.com/tblah created
https://github.com/llvm/llvm-project/pull/130788
This flag provides extra commentary in the assembly output. It is in
CodeGenOptions to match what is done in clang, even though the backend treats
it as a target option.
>From 5d3c1cc1d40b308b93cc47c78c6c3ca677f
https://github.com/tblah closed https://github.com/llvm/llvm-project/pull/129872
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tblah wrote:
> The comment mentions the additional edges might come through cancellations,
> but what test actually tests cancellation points?
Thanks for taking a look. Now I have added clang lit tests (including for
cancel) we can see that there are no changes beyond block names and removing
https://github.com/tblah updated
https://github.com/llvm/llvm-project/pull/129872
>From 3055b89b8ab937bfd4ba70d87e1733b05156d0ac Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Tue, 4 Mar 2025 17:19:53 +
Subject: [PATCH 1/2] [mlir][OpenMP] fix crash outlining infinite loop
Previously an ex
https://github.com/tblah approved this pull request.
LGTM thanks
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
https://github.com/tblah commented:
Thank you for up-streaming this
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
https://github.com/tblah approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/119718
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,155 @@
+
+
+# `DO CONCURRENT` 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 h
https://github.com/tblah 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
@@ -0,0 +1,155 @@
+
+
+# `DO CONCURRENT` 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 h
https://github.com/tblah approved this pull request.
Thanks for coming back to this @brad0
https://github.com/llvm/llvm-project/pull/125998
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-comm
https://github.com/tblah approved this pull request.
LGTM. As I understand it the test failures were fixed by
https://github.com/llvm/llvm-project/pull/123364
https://github.com/llvm/llvm-project/pull/123606
___
cfe-commits mailing list
cfe-commits@li
@@ -933,10 +933,12 @@ def O : Joined<["-"], "O">, Group,
def O_flag : Flag<["-"], "O">, Visibility<[ClangOption, CC1Option, FC1Option]>,
Alias, AliasArgs<["1"]>;
def Ofast : Joined<["-"], "Ofast">, Group,
- Visibility<[ClangOption, CC1Option, FlangOption]>,
- HelpTextForVar
https://github.com/tblah approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/119536
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tblah wrote:
> Hello! Did anyone notice a performance regression after this PR on exchange2?
> I see it on both EPYC 9684X and grace. The slowdown is more than 1.5x.
>
> I verified that performance restores when adding `-fwrapv` with the latest
> flang compiler. The compilation options are `-O
@@ -6980,3 +6980,37 @@ void driver::applyOverrideOptions(SmallVectorImpl &Args,
++S;
}
}
+
+/// Vectorize at all optimization levels greater than 1 except for -Oz.
+/// For -Oz the loop vectorizer is disabled, while the slp vectorizer is
+/// enabled.
+bool driver::shou
https://github.com/tblah approved this pull request.
Thanks for your patience and for bringing this back!
https://github.com/llvm/llvm-project/pull/118933
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
tblah wrote:
Hi @yus3710-fj, thanks for investigating the cause of the regression.
There was a recent patch which improved bwaves performance:
https://github.com/llvm/llvm-project/pull/111853. It seems that since this
patch, I no longer see a regression with the integer overflow enabled so I
https://github.com/tblah approved this pull request.
The code changes look good to me. I am not at all familiar with LoongArch64 so
it would be best if somebody else reviews that this implements the ABI
correctly.
https://github.com/llvm/llvm-project/pull/117108
___
@@ -1151,6 +1154,311 @@ struct TargetLoongArch64 : public
GenericTarget {
return GenericTarget::integerArgumentType(loc, argTy);
}
+
+ /// Flatten non-basic types, resulting in an array of types containing only
+ /// `IntegerType` and `FloatType`.
+ std::vector flatt
@@ -1151,6 +1154,311 @@ struct TargetLoongArch64 : public
GenericTarget {
return GenericTarget::integerArgumentType(loc, argTy);
}
+
+ /// Flatten non-basic types, resulting in an array of types containing only
+ /// `IntegerType` and `FloatType`.
+ std::vector flatt
@@ -1151,6 +1154,311 @@ struct TargetLoongArch64 : public
GenericTarget {
return GenericTarget::integerArgumentType(loc, argTy);
}
+
+ /// Flatten non-basic types, resulting in an array of types containing only
+ /// `IntegerType` and `FloatType`.
+ std::vector flatt
@@ -1151,6 +1154,311 @@ struct TargetLoongArch64 : public
GenericTarget {
return GenericTarget::integerArgumentType(loc, argTy);
}
+
+ /// Flatten non-basic types, resulting in an array of types containing only
+ /// `IntegerType` and `FloatType`.
+ std::vector flatt
@@ -1151,6 +1154,311 @@ struct TargetLoongArch64 : public
GenericTarget {
return GenericTarget::integerArgumentType(loc, argTy);
}
+
+ /// Flatten non-basic types, resulting in an array of types containing only
+ /// `IntegerType` and `FloatType`.
+ std::vector flatt
https://github.com/tblah approved this pull request.
This looks good to me but please don't merge without agreement on this approach
from the other reviewers.
https://github.com/llvm/llvm-project/pull/116432
___
cfe-commits mailing list
cfe-commits@li
@@ -490,6 +490,16 @@ void tools::AddLinkerInputs(const ToolChain &TC, const
InputInfoList &Inputs,
else
A.renderAsInput(Args, CmdArgs);
}
+ if (const Arg *A = Args.getLastArg(options::OPT_fveclib)) {
+if (A->getNumValues() == 1) {
+ StringRef V = A->getVa
@@ -490,6 +490,16 @@ void tools::AddLinkerInputs(const ToolChain &TC, const
InputInfoList &Inputs,
else
A.renderAsInput(Args, CmdArgs);
}
+ if (const Arg *A = Args.getLastArg(options::OPT_fveclib)) {
+if (A->getNumValues() == 1) {
+ StringRef V = A->getVa
https://github.com/tblah approved this pull request.
LGTM thanks
https://github.com/llvm/llvm-project/pull/113215
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah closed https://github.com/llvm/llvm-project/pull/113901
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah approved this pull request.
Thanks!
https://github.com/llvm/llvm-project/pull/113901
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tblah wrote:
@yus3710-fj after this patch we get a 7-10% slowdown for 503.bwaves_r from
SPEC2017. I built with `-Ofast -mcpu=native -fuse-ld=lld -flto`
As I understood it, the previous patch was supposed to enable your
transformation by default, and this one was only supposed to clean up old
https://github.com/tblah approved this pull request.
LGTM. Thanks for the extensive cleanup
https://github.com/llvm/llvm-project/pull/112533
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com
@@ -203,6 +203,30 @@ void Flang::AddAArch64TargetArgs(const ArgList &Args,
}
}
+void Flang::AddPPCTargetArgs(const ArgList &Args,
+ ArgStringList &CmdArgs) const {
+ bool VecExtabi = false;
+ for (const Arg *A : Args.filtered(options::OPT_mabi_E
@@ -1171,12 +1179,15 @@ void OpenMPIRBuilder::emitCancelationCheckImpl(Value
*CancelFlag,
// post finalization block that is known to the FiniCB callback.
Builder.SetInsertPoint(CancellationBlock);
if (ExitCB)
-ExitCB(Builder.saveIP());
+if (Error Err = ExitCB(Bu
@@ -1496,7 +1507,8 @@ IRBuilder<>::InsertPoint OpenMPIRBuilder::createParallel(
// Let the caller create the body.
assert(BodyGenCB && "Expected body generation callback!");
InsertPointTy CodeGenIP(PRegBodyBB, PRegBodyBB->begin());
- BodyGenCB(InnerAllocaIP, CodeGenIP);
https://github.com/tblah approved this pull request.
Thanks! LGTM
https://github.com/llvm/llvm-project/pull/110063
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah approved this pull request.
This looks good to me, thanks!
https://github.com/llvm/llvm-project/pull/110061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/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
@@ -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/tblah edited 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/tblah approved this pull request.
Thanks for this!
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/tblah approved this pull request.
LGTM thanks!
https://github.com/llvm/llvm-project/pull/112202
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah approved this pull request.
Thanks!
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
@@ -787,6 +787,9 @@ void Flang::ConstructJob(Compilation &C, const JobAction
&JA,
if (Args.hasArg(options::OPT_fopenmp_force_usm))
CmdArgs.push_back("-fopenmp-force-usm");
+ // TODO: OpenMP support isn't "done" yet, so for now we warn that it
+ // is e
https://github.com/tblah approved this pull request.
LGTM but if possible please wait for another reviewer
https://github.com/llvm/llvm-project/pull/110598
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/l
tblah wrote:
We discussed this further offline and I am now convinced there could be a valid
usecase for this. Some C++ code might use header-only parts of the C++ standard
library. Setting `-stdlib=` will effect the include paths clang uses for stdlib
when compiling.
When flang links the pro
tblah wrote:
If this is completely ignored and does not effect the link line, are you sure
the resulting binary actually uses the right c++ library?
https://github.com/llvm/llvm-project/pull/110598
___
cfe-commits mailing list
cfe-commits@lists.llvm.o
@@ -0,0 +1,48 @@
+// REQUIRES: system-linux
tblah wrote:
I'm not sure about this test. Usually we would go about testing something like
this by running FileCheck on the output of `flang-new -###` to see what is
passed to the linker (and that the linker gets cal
tblah wrote:
I don't think it is correct to say that the option was ignored. It may still be
read by the compiler driver (which generates the link line). In fact it
probably *should* be processed by the compiler driver in order to set the C++
standard library correctly - otherwise you could ju
tblah wrote:
The CI failures look relevant. The problem isn't immediately apparent to me
from the diff (which looks good to me).
https://github.com/llvm/llvm-project/pull/109907
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
@@ -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
https://github.com/tblah edited 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
https://github.com/tblah approved this pull request.
LGTM
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
https://github.com/tblah commented:
Is `-fdiagnostics-color=auto` too host-environment dependent to test?
https://github.com/llvm/llvm-project/pull/109210
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/
https://github.com/tblah approved this pull request.
LGTM, thanks
https://github.com/llvm/llvm-project/pull/108868
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah approved this pull request.
Thanks for the updates. LGTM
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/
@@ -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"
tblah wrote:
nit: maybe I'm being overly pedantic here, but I think the meaning of the test
woul
https://github.com/tblah approved this pull request.
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/tblah approved this pull request.
Thanks!
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
https://github.com/tblah commented:
Thanks for the quick updates. I don't have any idea off the top of my head how
to fix the LLVM timings. Do any other reviewers have thoughts/experience in
this area?
https://github.com/llvm/llvm-project/pull/107270
___
@@ -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
@@ -4,6 +4,7 @@ add_subdirectory(Decimal)
add_subdirectory(Lower)
add_subdirectory(Parser)
add_subdirectory(Semantics)
+add_subdirectory(Support)
tblah wrote:
Why did you decide to put this in a new subdirectory and not just part of
Frontend? Is there a plan
@@ -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
@@ -0,0 +1,18 @@
+! Check that -ftime-report flag is passed as-is to fc1. The value of the flag
+! is only checked there. This behavior intentionally mirrors that of clang.
+!
+! -ftime-report= is currently not supported because we do not support detailed
+! timing information on
https://github.com/tblah commented:
Thanks for working on this
https://github.com/llvm/llvm-project/pull/107270
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah edited https://github.com/llvm/llvm-project/pull/107270
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -254,6 +276,43 @@ class CompilerInstance {
/// Produces the string which represents target feature
std::string getTargetFeatures();
+ /// {
+ /// @name Timing
+ /// @{
+ bool isTimingEnabled() { return timingMgr.isEnabled(); }
tblah wrote:
nit: Do
https://github.com/tblah approved this pull request.
I agree with other reviewers that a generic option would be better than a list
of particular types. But for me that's just a nit because these are hidden
debug options anyway
https://github.com/llvm/llvm-project/pull/107126
_
https://github.com/tblah closed https://github.com/llvm/llvm-project/pull/107087
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah updated
https://github.com/llvm/llvm-project/pull/107087
>From 10a0bc5c82ef820593528c034c26cd1f5c90d6e5 Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Tue, 3 Sep 2024 11:11:47 +
Subject: [PATCH 1/3] [flang][Driver] support -fno-openmp
Closes #83148
---
clang/incl
tblah wrote:
> As I'm not seeing how linker is invoked, I just need to ask (for the sake of
> completeness) will it also prevent addition of `-lomp` to the linker's
> command line?
Thanks for pointing this out. I've added a test for the linker invocation.
https://github.com/llvm/llvm-project/
https://github.com/tblah updated
https://github.com/llvm/llvm-project/pull/107087
>From 10a0bc5c82ef820593528c034c26cd1f5c90d6e5 Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Tue, 3 Sep 2024 11:11:47 +
Subject: [PATCH 1/2] [flang][Driver] support -fno-openmp
Closes #83148
---
clang/incl
https://github.com/tblah created
https://github.com/llvm/llvm-project/pull/107087
Closes #83148
>From 10a0bc5c82ef820593528c034c26cd1f5c90d6e5 Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Tue, 3 Sep 2024 11:11:47 +
Subject: [PATCH] [flang][Driver] support -fno-openmp
Closes #83148
---
https://github.com/tblah approved this pull request.
Code changes look good to me. I can't speak to the needs of anyone else using
offloading.
I see this is copied exactly from `Toolchains/Clang.cpp`. I think that's okay
for such a small bit of code.
https://github.com/llvm/llvm-project/pull/
https://github.com/tblah approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/99058
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah approved this pull request.
LGTM. Thanks for picking this up Mats.
Ideally, wait for @Thirumalai-Shaktivel to have another look before merging.
https://github.com/llvm/llvm-project/pull/95411
___
cfe-commits mailing list
cfe-c
@@ -21,4 +22,14 @@ CodeGenOptions::CodeGenOptions() {
#include "flang/Frontend/CodeGenOptions.def"
}
+std::optional getCodeModel(llvm::StringRef string) {
+ return llvm::StringSwitch>(string)
+ .Case("tiny", llvm::CodeModel::Model::Tiny)
+ .Case("small", llvm::CodeM
https://github.com/tblah approved this pull request.
I think this is okay. I don't want to block the PR on diagnostics for an FC1
flag which aren't present in clang. LGTM and thanks for answering my questions.
Please also wait for @banach-space's approval.
https://github.com/llvm/llvm-project/
https://github.com/tblah approved this pull request.
@DavidTruby is on holiday
We have tested this on our end and it does not re-introduce the issue. Thanks
for the revert & fix!
https://github.com/llvm/llvm-project/pull/96799
___
cfe-commits mailing
@@ -333,6 +333,9 @@ void Flang::AddAMDGPUTargetArgs(const ArgList &Args,
StringRef Val = A->getValue();
CmdArgs.push_back(Args.MakeArgString("-mcode-object-version=" + Val));
}
+
+ const ToolChain &TC = getToolChain();
+ TC.addClangTargetOptions(Args, CmdArgs, Actio
1 - 100 of 270 matches
Mail list logo
