vzakhari wrote:
Thank you for the pointers and the information, Tom! It looks like exchange2
is pretty much the same problem as with bwaves. I posted a note into
https://github.com/llvm/llvm-project/issues/117318
https://github.com/llvm/llvm-project/pull/110063
__
https://github.com/vzakhari closed
https://github.com/llvm/llvm-project/pull/120320
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vzakhari wrote:
Okay, I do not see an easy way to allow this option for `clang` without
printing it in the `--help` output, so I guess I will let it behave same way as
the other Flang-only options.
https://github.com/llvm/llvm-project/pull/120320
___
@@ -3465,7 +3465,7 @@ defm diagnostics_show_line_numbers :
BoolFOption<"diagnostics-show-line-numbers"
PosFlag>;
def fno_realloc_lhs : Flag<["-"], "fno-realloc-lhs">, Group,
HelpText<"An allocatable left-hand side of an intrinsic assignment is
assumed to be allocated and
vzakhari wrote:
> > After #120165 clang started complaining about unknown option
> > -f[no-]realloc-lhs.
>
> Was this on an existing test?
I noticed it when trying to compile a Fortran/C mix test where flang/clang were
used as compilers and I passed `-fno-realloc-lhs` in flags that were used
https://github.com/vzakhari created
https://github.com/llvm/llvm-project/pull/120320
After #120165 clang started complaining about unknown
option -f[no-]realloc-lhs. This change fixes it to ignore
the option like it used to be.
>From 5d8d3e08eb322b72f4058b98fc7ea0d6321eaa6c Mon Sep 17 00:00:00
https://github.com/vzakhari closed
https://github.com/llvm/llvm-project/pull/120165
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vzakhari wrote:
> Just a comment, no action required. I see some hits for realloc-lhs in the
> gfortran testsuite. Will some of these tests start behaving differently?
Thank you for the heads up, Kiran! I did not change `-Wrealloc-lhs`, so they
should behave the same way. Those that use `-fno-
https://github.com/vzakhari created
https://github.com/llvm/llvm-project/pull/120165
-frealloc-lhs is the default.
If -fno-realloc-lhs is specified, then an allocatable on the left
side of an intrinsic assignment is not implicitly (re)allocated
to conform with the right hand side. Fortran runtim
https://github.com/vzakhari approved this pull request.
Looks good to me, but maybe we can have just a single option that accepts
predefined words like `real2`, `integer4`, etc. I wonder if we can process
multiple instances of an option like
https://github.com/llvm/llvm-project/blob/0ad6cee92
@@ -322,24 +306,20 @@ struct hash_state {
}
};
-
-/// A global, fixed seed-override variable.
-///
-/// This variable can be set using the \see llvm::set_fixed_execution_seed
-/// function. See that function for details. Do not, under any circumstances,
-/// set or read this
@@ -322,24 +306,20 @@ struct hash_state {
}
};
-
-/// A global, fixed seed-override variable.
-///
-/// This variable can be set using the \see llvm::set_fixed_execution_seed
-/// function. See that function for details. Do not, under any circumstances,
-/// set or read this
https://github.com/vzakhari approved this pull request.
It looks good to me except maybe for the naming of the lowering option. Thank
you for doing this!
https://github.com/llvm/llvm-project/pull/91579
___
cfe-commits mailing list
cfe-commits@lists.l
https://github.com/vzakhari edited
https://github.com/llvm/llvm-project/pull/91579
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -34,5 +34,9 @@ ENUM_LOWERINGOPT(NoPPCNativeVecElemOrder, unsigned, 1, 0)
/// On by default.
ENUM_LOWERINGOPT(Underscoring, unsigned, 1, 1)
+/// If true, add nsw flags to arithmetic operations for integer.
+/// Off by default.
+ENUM_LOWERINGOPT(NoSignedWrap, unsigned, 1, 0)
vzakhari wrote:
FYI, it looks like this change broke `compiler-rt` build, e.g. in
https://lab.llvm.org/buildbot/#/builders/270/builds/12485
https://github.com/llvm/llvm-project/pull/87684
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
@@ -6488,6 +6488,9 @@ defm stack_arrays : BoolOptionWithoutMarshalling<"f",
"stack-arrays",
defm loop_versioning : BoolOptionWithoutMarshalling<"f",
"version-loops-for-stride",
PosFlag,
NegFlag>;
+
+def fcuda : Flag<["-"], "fcuda">, Group,
vzakhari wrote
https://github.com/vzakhari edited
https://github.com/llvm/llvm-project/pull/84944
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vzakhari approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/84944
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vzakhari wrote:
I do not think the Windows CI failure has anything to do with my changes. I am
going to merge this.
https://github.com/llvm/llvm-project/pull/82832
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin
https://github.com/vzakhari closed
https://github.com/llvm/llvm-project/pull/82832
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vzakhari updated
https://github.com/llvm/llvm-project/pull/82832
>From ef2b9e87295688743771f1a64b15aa6fee6499df Mon Sep 17 00:00:00 2001
From: Slava Zakharin
Date: Fri, 23 Feb 2024 13:08:49 -0800
Subject: [PATCH 1/2] [flang] Fixes for LIT testing of
FLANG_RUNTIME_F128_MATH_L
https://github.com/vzakhari updated
https://github.com/llvm/llvm-project/pull/82832
>From 4ad0b005081fe2286970b4c22721fe72ed26cf8b Mon Sep 17 00:00:00 2001
From: Slava Zakharin
Date: Fri, 23 Feb 2024 13:08:49 -0800
Subject: [PATCH 1/2] [flang] Fixes for LIT testing of
FLANG_RUNTIME_F128_MATH_L
https://github.com/vzakhari updated
https://github.com/llvm/llvm-project/pull/82832
>From fea6f95d3cfaa3ae75e0f8312198a20c36e79ad7 Mon Sep 17 00:00:00 2001
From: Slava Zakharin
Date: Fri, 23 Feb 2024 13:08:49 -0800
Subject: [PATCH 1/2] [flang] Fixes for LIT testing of
FLANG_RUNTIME_F128_MATH_L
https://github.com/vzakhari updated
https://github.com/llvm/llvm-project/pull/82832
>From 81ea8a2cf3a1cb4b9e99d590f7a80b156fe609b8 Mon Sep 17 00:00:00 2001
From: Slava Zakharin
Date: Fri, 23 Feb 2024 13:08:49 -0800
Subject: [PATCH 1/2] [flang] Fixes for LIT testing of
FLANG_RUNTIME_F128_MATH_L
https://github.com/vzakhari created
https://github.com/llvm/llvm-project/pull/82832
Follow-up for #81971 to fix the disabled LIT test and add
LIT tests for lowering of the added math intrinsics.
>From 81ea8a2cf3a1cb4b9e99d590f7a80b156fe609b8 Mon Sep 17 00:00:00 2001
From: Slava Zakharin
Date:
vzakhari wrote:
I have been doing some renaming and restructuring before uploading this for
review, and I did not test the final version with `libquadmath`. The latest
commit makes the `-DFLANG_RUNTIME_F128_MATH_LIB=libquadmath` compiler fully
functional (the end-to-end test is working).
Tha
https://github.com/vzakhari updated
https://github.com/llvm/llvm-project/pull/81971
>From c798a2b74df57a1375882fb13a88ccf946f4bfbf Mon Sep 17 00:00:00 2001
From: Slava Zakharin
Date: Thu, 15 Feb 2024 20:01:35 -0800
Subject: [PATCH 1/3] [RFC][flang][runtime] Add FortranFloat128Math wrapper
libr
https://github.com/vzakhari updated
https://github.com/llvm/llvm-project/pull/81971
>From c798a2b74df57a1375882fb13a88ccf946f4bfbf Mon Sep 17 00:00:00 2001
From: Slava Zakharin
Date: Thu, 15 Feb 2024 20:01:35 -0800
Subject: [PATCH 1/2] [RFC][flang][runtime] Add FortranFloat128Math wrapper
libr
https://github.com/vzakhari updated
https://github.com/llvm/llvm-project/pull/81971
>From 32a034ada7ba4d5e4c195b5ef36cf671dcc2e06a Mon Sep 17 00:00:00 2001
From: Slava Zakharin
Date: Thu, 15 Feb 2024 20:01:35 -0800
Subject: [PATCH 1/2] [RFC][flang][runtime] Add FortranFloat128Math wrapper
libr
@@ -657,10 +657,61 @@ static llvm::cl::opt
"instead of libm complex operations"),
llvm::cl::init(false));
+/// Return a string containing the given Fortran intrinsic name
+/// with the type of its arguments specified in
https://github.com/vzakhari updated
https://github.com/llvm/llvm-project/pull/81971
>From 32a034ada7ba4d5e4c195b5ef36cf671dcc2e06a Mon Sep 17 00:00:00 2001
From: Slava Zakharin
Date: Thu, 15 Feb 2024 20:01:35 -0800
Subject: [PATCH] [RFC][flang][runtime] Add FortranFloat128Math wrapper
library.
https://github.com/vzakhari created
https://github.com/llvm/llvm-project/pull/81971
Implemented few entry points for REAL(16) math in FortranF128Math
static library. It is a thin wrapper around GNU libquadmath.
Flang driver can always link it, and the dependencies will
be brought in as needed.
T
https://github.com/vzakhari commented:
Thank you, Tom!
https://github.com/llvm/llvm-project/pull/74250
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -142,6 +142,26 @@ void Flang::addCodegenOptions(const ArgList &Args,
if (shouldLoopVersion(Args))
CmdArgs.push_back("-fversion-loops-for-stride");
+ Arg *aliasAnalysis = Args.getLastArg(options::OPT_falias_analysis,
+ options::OP
https://github.com/vzakhari commented:
Thank you for the changes, Tom!
I have one minor comment, but I would like to ask to merge this after US
holidays, if possible. Could you please postpone the merging until Monday GMT?
https://github.com/llvm/llvm-project/pull/73111
__
https://github.com/vzakhari edited
https://github.com/llvm/llvm-project/pull/73111
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vzakhari approved this pull request.
Thank you, Jean!
https://github.com/llvm/llvm-project/pull/71820
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vzakhari wrote:
Hi @tblah, is this ready for a merge? Let me know if you are just waiting for
a review.
https://github.com/llvm/llvm-project/pull/68597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
vzakhari wrote:
Hello @egorzhdan, this change breaks
https://lab.llvm.org/buildbot/#/builders/270/builds/2125. Could you please fix
or revert?
https://github.com/llvm/llvm-project/pull/70827
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ht
40 matches
Mail list logo
