https://github.com/banach-space closed
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
banach-space wrote:
Closing in favour of #110023
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
h-vetinari wrote:
Given the state of the discussion on discourse, I looked at this PR again, and
was curious about conflicts with main. I don't think I'm the right person to
propose this as a PR, but if someone wants to take my attempt as a jumping-off
point, I put the branch
[here](https://g
brad0 wrote:
> ok, but is there some threshold of percent that pass, or a specific list that
> we want to be sure pass? My understanding is that we've been running (at
> least a subset of) the gfortran test suite for a while now. What makes the
> current results not sufficient?
I think it's m
sscalpone wrote:
Still gathering feedback
https://discourse.llvm.org/t/proposal-rename-flang-new-to-flang/69462/54
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
h-vetinari wrote:
It appears this PR now has conflicts that need resolving...
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/banach-space updated
https://github.com/llvm/llvm-project/pull/74377
From 1eefb9136ea042e44ada4be66a18b4bf69c18fe3 Mon Sep 17 00:00:00 2001
From: Andrzej Warzynski
Date: Sat, 2 Dec 2023 14:01:02 +
Subject: [PATCH 1/4] [flang][driver] Rename `flang-new` as `flang`
This pa
@@ -1,15 +1,15 @@
-! Test that flang-new OpenMP and OpenMP offload related
+! Test that flang OpenMP and OpenMP offload related
! commands forward or expand to the appropriate commands
-! for flang-new -fc1 as expected. Assumes a gfx90a, aarch64,
+! for flang -fc1 as expected.
@@ -15,17 +15,13 @@ local:
```
There are two main drivers in Flang:
banach-space wrote:
Good point 👍🏻
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
https://github.com/banach-space edited
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/banach-space edited
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -193,26 +190,29 @@ driver, `clang -cc1` and consists of the following
classes:
This list is not exhaustive and only covers the main classes that implement the
driver. The main entry point for the frontend driver, `fc1_main`, is
implemented in `flang/tools/flang-driver/drive
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/h-vetinari edited
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/h-vetinari commented:
Some drive-by comments - very happy to see this happening!
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
mjklemm wrote:
I have a Windows test machine. How can I help?
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/banach-space updated
https://github.com/llvm/llvm-project/pull/74377
From 1eefb9136ea042e44ada4be66a18b4bf69c18fe3 Mon Sep 17 00:00:00 2001
From: Andrzej Warzynski
Date: Sat, 2 Dec 2023 14:01:02 +
Subject: [PATCH 1/2] [flang][driver] Rename `flang-new` as `flang`
This pa
https://github.com/banach-space updated
https://github.com/llvm/llvm-project/pull/74377
From 1eefb9136ea042e44ada4be66a18b4bf69c18fe3 Mon Sep 17 00:00:00 2001
From: Andrzej Warzynski
Date: Sat, 2 Dec 2023 14:01:02 +
Subject: [PATCH 1/2] [flang][driver] Rename `flang-new` as `flang`
This pa
kiranchandramohan wrote:
I am assuming you have got a go-ahead from the code owner and others involved
in the discourse discussion. It is probably good to post a link to this patch
in the relevant discourse post for information and confirmation.
https://github.com/llvm/llvm-project/pull/74377
tschuett wrote:
Are you going to add an entry to the [Release
Nodes](https://github.com/llvm/llvm-project/blob/main/flang/docs/ReleaseNotes.md)?
https://github.com/llvm/llvm-project/pull/74377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ht
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-clang
Author: Andrzej Warzyński (banach-space)
Changes
This patch renames `flang-new` as `flang`. Similarly to Clang, Flang's
driver executable will be called:
* `flang-`
A symlink called `flang` point
21 matches
Mail list logo