Author: Florian Mayer
Date: 2024-01-16T16:37:21-08:00
New Revision: bcdbff65490a76f30624ded5de5e6301e28f1b96
URL:
https://github.com/llvm/llvm-project/commit/bcdbff65490a76f30624ded5de5e6301e28f1b96
DIFF:
https://github.com/llvm/llvm-project/commit/bcdbff65490a76f30624ded5de5e6301e28f1b96.diff
https://github.com/mordante updated
https://github.com/llvm/llvm-project/pull/76246
>From eebe9b2fab5c5eef4776852270bf70af4626cfcb Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Wed, 20 Dec 2023 20:43:38 +0100
Subject: [PATCH 1/3] [libc++][modules] Adds module testing.
This adds a new
@@ -52,6 +52,21 @@ def _executeScriptInternal(test, litConfig, commands):
return (out, err, exitCode, timeoutInfo, parsedCommands)
+def _validateModuleDependencies(modules):
+for m in modules:
+if m not in ("std", "std.compat"):
+raise
https://github.com/mordante updated
https://github.com/llvm/llvm-project/pull/76246
>From eebe9b2fab5c5eef4776852270bf70af4626cfcb Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Wed, 20 Dec 2023 20:43:38 +0100
Subject: [PATCH 1/2] [libc++][modules] Adds module testing.
This adds a new
https://github.com/llvm-beanz approved this pull request.
Looks like you angered the clang-format god, but otherwise LGTM.
https://github.com/llvm/llvm-project/pull/78225
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
@@ -131,13 +138,65 @@ def parseScript(test, preamble):
script += preamble
script += scriptInTest
+has_std_module = False
+has_std_compat_module = False
+for module in modules:
+if module == "std":
+has_std_module = True
+elif
@@ -132,13 +154,62 @@ def parseScript(test, preamble):
script += scriptInTest
# Add compile flags specified with ADDITIONAL_COMPILE_FLAGS.
+# Modules need to be build with the same compilation flags as the
+# test. So add these flags before adding the modules.
@@ -132,13 +154,62 @@ def parseScript(test, preamble):
script += scriptInTest
# Add compile flags specified with ADDITIONAL_COMPILE_FLAGS.
+# Modules need to be build with the same compilation flags as the
+# test. So add these flags before adding the modules.
@@ -132,13 +154,62 @@ def parseScript(test, preamble):
script += scriptInTest
# Add compile flags specified with ADDITIONAL_COMPILE_FLAGS.
+# Modules need to be build with the same compilation flags as the
+# test. So add these flags before adding the modules.
@@ -132,13 +154,62 @@ def parseScript(test, preamble):
script += scriptInTest
# Add compile flags specified with ADDITIONAL_COMPILE_FLAGS.
+# Modules need to be build with the same compilation flags as the
+# test. So add these flags before adding the modules.
@@ -52,6 +52,21 @@ def _executeScriptInternal(test, litConfig, commands):
return (out, err, exitCode, timeoutInfo, parsedCommands)
+def _validateModuleDependencies(modules):
+for m in modules:
+if m not in ("std", "std.compat"):
+raise
@@ -0,0 +1,22 @@
+//===--===//
+//
+// 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:
@@ -132,13 +154,62 @@ def parseScript(test, preamble):
script += scriptInTest
# Add compile flags specified with ADDITIONAL_COMPILE_FLAGS.
+# Modules need to be build with the same compilation flags as the
+# test. So add these flags before adding the modules.
https://github.com/ldionne approved this pull request.
LGTM with some comments, thanks!
https://github.com/llvm/llvm-project/pull/76246
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
@@ -0,0 +1,24 @@
+//===--===//
+//
+// 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:
@@ -0,0 +1,24 @@
+//===--===//
+//
+// 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:
@@ -132,13 +154,62 @@ def parseScript(test, preamble):
script += scriptInTest
# Add compile flags specified with ADDITIONAL_COMPILE_FLAGS.
+# Modules need to be build with the same compilation flags as the
+# test. So add these flags before adding the modules.
@@ -132,13 +154,62 @@ def parseScript(test, preamble):
script += scriptInTest
# Add compile flags specified with ADDITIONAL_COMPILE_FLAGS.
+# Modules need to be build with the same compilation flags as the
ldionne wrote:
```suggestion
#
https://github.com/ldionne edited
https://github.com/llvm/llvm-project/pull/76246
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/kparzysz edited
https://github.com/llvm/llvm-project/pull/78289
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -4290,6 +4291,18 @@ class FirConverter : public
Fortran::lower::AbstractConverter {
assert(blockId == 0 && "invalid blockId");
assert(activeConstructStack.empty() && "invalid construct stack state");
+// Set target_cpu and target_features attributes to be
https://github.com/kiranchandramohan approved this pull request.
LGTM. Please wait for @DominikAdamski
Not for this patch, but would we need to add this info for all outlined
functions?
https://github.com/llvm/llvm-project/pull/78291
___
@@ -4290,6 +4291,18 @@ class FirConverter : public
Fortran::lower::AbstractConverter {
assert(blockId == 0 && "invalid blockId");
assert(activeConstructStack.empty() && "invalid construct stack state");
+// Set target_cpu and target_features attributes to be
https://github.com/philnik777 approved this pull request.
I'm OK with this as a quick fix, since Mark promised to fix it properly in the
long term.
https://github.com/llvm/llvm-project/pull/76268
___
llvm-branch-commits mailing list
llvmbot wrote:
@llvm/pr-subscribers-mlir-openmp
Author: Sergio Afonso (skatrak)
Changes
This patch removes the omp.target module attribute, since the information it
held on the target CPU and features is available to each function as a function
attribute. Target outlining during the
llvmbot wrote:
@llvm/pr-subscribers-flang-driver
@llvm/pr-subscribers-flang-fir-hlfir
@llvm/pr-subscribers-mlir
Author: Sergio Afonso (skatrak)
Changes
This patch removes the omp.target module attribute, since the information it
held on the target CPU and features is available to each
https://github.com/skatrak created
https://github.com/llvm/llvm-project/pull/78291
This patch removes the omp.target module attribute, since the information it
held on the target CPU and features is available to each function as a function
attribute. Target outlining during the MLIR to LLVM
llvmbot wrote:
@llvm/pr-subscribers-flang-driver
@llvm/pr-subscribers-flang-fir-hlfir
Author: Sergio Afonso (skatrak)
Changes
This patch forwards the target CPU and features information from the Flang
frontend to MLIR func.func operation attributes, which are later used to
populate the
https://github.com/skatrak created
https://github.com/llvm/llvm-project/pull/78289
This patch forwards the target CPU and features information from the Flang
frontend to MLIR func.func operation attributes, which are later used to
populate the target_cpu and target_features llvm.func
koachan wrote:
Okay, new PR is at https://github.com/llvm/llvm-project/pull/78280.
https://github.com/llvm/llvm-project/pull/77196
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/77761
>From 1b5524ae8874e389d373a55417919afa56beb2b5 Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Mon, 8 Jan 2024 15:53:07 -0600
Subject: [PATCH 1/4] [Flang][OpenMP] Restructure recursive lowering in
https://github.com/legrosbuffle approved this pull request.
https://github.com/llvm/llvm-project/pull/76653
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
32 matches
Mail list logo
