From 03e520c55f38f7b01ddfc2919bb7ebce3e236357 Mon Sep 17 00:00:00 2001
From: agozillon <Andrew.Gozillon@amd.com>
Date: Tue, 13 Feb 2024 08:23:53 -0600
Subject: [PATCH] forgot to include the always assert removal...

---
 flang/lib/Lower/OpenMP.cpp | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/flang/lib/Lower/OpenMP.cpp b/flang/lib/Lower/OpenMP.cpp
index 52928ab8f11ebd..63a527704e724a 100644
--- a/flang/lib/Lower/OpenMP.cpp
+++ b/flang/lib/Lower/OpenMP.cpp
@@ -2064,16 +2064,8 @@ bool ClauseProcessor::processMap(
           }
 
           if (std::get<std::optional<Fortran::parser::OmpMapType::Always>>(
-                  oMapType->t)) {
-            // NOTE from Andrew Gozillon: Temporary TODO assert at request of
-            // Brian from Apps team to better check where always is used in an
-            // App as it's only supported in the sense that we add the flag at
-            // the moment. Remove when we begin to support always fully. I have
-            // also had to deactive a few tests inside of
-            // Lower/OpenMP/target.f90 & Lower/OpenMP/FIR/target.f90
-            TODO(currentLocation, "OMP_MAP_ALWAYS");
+                  oMapType->t))
             mapTypeBits |= llvm::omp::OpenMPOffloadMappingFlags::OMP_MAP_ALWAYS;
-          }
         } else {
           mapTypeBits |= llvm::omp::OpenMPOffloadMappingFlags::OMP_MAP_TO |
                          llvm::omp::OpenMPOffloadMappingFlags::OMP_MAP_FROM;