[all-commits] [llvm/llvm-project] 0163ac: [Flang][OpenMP]Add tests for TODOs and small chang...

[Mats Petersson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0163ac1f53abc0a0f6e5b7e56912c1dee67e7f32
      https://github.com/llvm/llvm-project/commit/0163ac1f53abc0a0f6e5b7e56912c1dee67e7f32
  Author: Mats Petersson <mats.petersson at arm.com>
  Date:   2024-10-11 (Fri, 11 Oct 2024)

  Changed paths:
    M flang/lib/Lower/OpenMP/OpenMP.cpp
    A flang/test/Lower/OpenMP/Todo/reduction-inscan.f90
    A flang/test/Lower/OpenMP/Todo/reduction-task.f90
    A flang/test/Lower/OpenMP/Todo/target-inreduction.f90
    A flang/test/Lower/OpenMP/Todo/task-inreduction.f90
    M flang/test/Lower/OpenMP/Todo/task_mergeable.f90
    A flang/test/Lower/OpenMP/Todo/taskgroup-task-reduction.f90
    A flang/test/Lower/OpenMP/Todo/taskloop.f90
    A flang/test/Lower/OpenMP/Todo/taskwait-depend.f90
    A flang/test/Lower/OpenMP/Todo/taskwait-nowait.f90

  Log Message:
  -----------
  [Flang][OpenMP]Add tests for TODOs and small changes to improve messages (#111562)

The bulk of this change are new tests to check that we get a "Not yet
implemneted: *some stuff here*" message when using some not yet
supported OpenMP functionality.

For some of these cases, this also means adding additional clauses to a
filter list in OpenMP.cpp - this changes nothing [to the best of my
understanding] other than allowing the clause to get to the point where
it can be rejected in a TODO with a more clear message. One of the TOOD
filters were missing Mergeable clause, so this was also added and the
existing test updated for the new more specific error message.

There is no functional change intended here.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications