[flang-commits] [flang] [flang][cuda] Remove extra managed spelling from bbc (PR #175803)
Valentin Clement バレンタイン クレメン via flang-commits
flang-commits at lists.llvm.org
Tue Jan 13 09:46:40 PST 2026
https://github.com/clementval created https://github.com/llvm/llvm-project/pull/175803
`bbc` is just a testing tool so we don't need to support multiple option spelling. Keep it simple.
>From 81698c93b03c3f1e33238c56f64d113a73b143db Mon Sep 17 00:00:00 2001
From: Valentin Clement <clementval at gmail.com>
Date: Tue, 13 Jan 2026 09:45:44 -0800
Subject: [PATCH] [flang][cuda] Remove extra managed spelling from bbc
---
flang/lib/Semantics/resolve-names.cpp | 2 +-
flang/test/Lower/CUDA/cuda-gpu-managed.cuf | 2 +-
flang/tools/bbc/bbc.cpp | 5 ++---
3 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/flang/lib/Semantics/resolve-names.cpp b/flang/lib/Semantics/resolve-names.cpp
index d2a3331710dba..310f7ab97e9a0 100644
--- a/flang/lib/Semantics/resolve-names.cpp
+++ b/flang/lib/Semantics/resolve-names.cpp
@@ -9913,7 +9913,7 @@ void ResolveNamesVisitor::FinishSpecificationPart(
SetBindNameOn(symbol);
}
}
- // -gpu=mem:managed: implicitly treat allocatable arrays as managed.
+ // Implicitly treat allocatable arrays as managed when feature is enabled.
// This is done after all explicit CUDA attributes have been processed.
if (context().languageFeatures().IsEnabled(
common::LanguageFeature::CudaManaged))
diff --git a/flang/test/Lower/CUDA/cuda-gpu-managed.cuf b/flang/test/Lower/CUDA/cuda-gpu-managed.cuf
index b015b470b921b..6c69f4d71674c 100644
--- a/flang/test/Lower/CUDA/cuda-gpu-managed.cuf
+++ b/flang/test/Lower/CUDA/cuda-gpu-managed.cuf
@@ -1,4 +1,4 @@
-! RUN: bbc -emit-hlfir -fcuda -gpu=mem:managed %s -o - | FileCheck %s
+! RUN: bbc -emit-hlfir -fcuda -gpu=managed %s -o - | FileCheck %s
! Test -gpu=managed flag: allocatable arrays without explicit CUDA attributes
! should be implicitly treated as managed.
diff --git a/flang/tools/bbc/bbc.cpp b/flang/tools/bbc/bbc.cpp
index aefb4369fc761..0b87dd634af1e 100644
--- a/flang/tools/bbc/bbc.cpp
+++ b/flang/tools/bbc/bbc.cpp
@@ -638,11 +638,10 @@ int main(int argc, char **argv) {
Fortran::common::LanguageFeature::CudaWarpMatchFunction, false);
}
- if (enableGPUMode == "managed" || enableGPUMode == "mem:managed") {
+ if (enableGPUMode == "managed")
options.features.Enable(Fortran::common::LanguageFeature::CudaManaged);
- } else if (enableGPUMode == "unified" || enableGPUMode == "mem:unified") {
+ else if (enableGPUMode == "unified")
options.features.Enable(Fortran::common::LanguageFeature::CudaUnified);
- }
if (fixedForm) {
options.isFixedForm = fixedForm;
More information about the flang-commits
mailing list