[flang-commits] [flang] [flang][OpenMP] Add lowering for assume and assumes directives (PR #205615)

[via flang-commits]

================
@@ -4205,8 +4206,8 @@ static void genOMP(lower::AbstractConverter &converter, lower::SymMap &symTable,
                    semantics::SemanticsContext &semaCtx,
                    lower::pft::Evaluation &eval,
                    const parser::OmpAssumesDirective &assumesConstruct) {
-  if (!semaCtx.langOptions().OpenMPSimd)
-    TODO(converter.getCurrentLocation(), "OpenMP ASSUMES declaration");
+  // Assumption clauses are hints with no representation in the OpenMP dialect,
+  // so this declarative directive is a no-op.
----------------
Ritanya-B-Bharadwaj wrote:

Agreed. For now I've replaced the silent no-op with an explicit "not yet implemented." since this was just the frontend patch. Can i take this up in the follow-up PR?

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