Skip to content

Commit

Permalink
forgot to include the always assert removal...
Browse files Browse the repository at this point in the history
  • Loading branch information
agozillon committed Feb 13, 2024
1 parent d24bde8 commit 03e520c
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions flang/lib/Lower/OpenMP.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down

0 comments on commit 03e520c

Please sign in to comment.