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
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
@@ -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
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://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
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
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
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
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
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).
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
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
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
@@ -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
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.
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 ,
if (shouldLoopVersion(Args))
CmdArgs.push_back("-fversion-loops-for-stride");
+ Arg *aliasAnalysis = Args.getLastArg(options::OPT_falias_analysis,
+
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
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
28 matches
Mail list logo