[llvm-branch-commits] [flang] [Flang][OpenMP] Push genEval calls to individual operations, NFC (PR #77758)

Sergio Afonso via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Fri Jan 12 05:13:11 PST 2024


================
@@ -3655,8 +3661,7 @@ void Fortran::lower::genOpenMPDeclarativeConstruct(
     Fortran::lower::pft::Evaluation &eval,
     const Fortran::parser::OpenMPDeclarativeConstruct &omp) {
   genOMP(converter, eval, omp);
----------------
skatrak wrote:

Nit: I guess the `symTable` argument wasn't added here because it wasn't currently needed by any of the lowering functions for OpenMP declarative constructs. However, I think it would be good to still add it so that all `genOMP` functions have the same signature, even if just for the sake of uniformity.

Feel free to ignore if you disagree.

https://github.com/llvm/llvm-project/pull/77758


More information about the llvm-branch-commits mailing list