public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/99666] New: [OpenMP][5.0] Support 'affinity' clause in 'omp task'
@ 2021-03-19 11:32 burnus at gcc dot gnu.org
  2021-03-19 11:35 ` [Bug middle-end/99666] " jakub at gcc dot gnu.org
  2023-07-14 10:05 ` burnus at gcc dot gnu.org
  0 siblings, 2 replies; 3+ messages in thread
From: burnus at gcc dot gnu.org @ 2021-03-19 11:32 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99666

            Bug ID: 99666
           Summary: [OpenMP][5.0] Support 'affinity' clause in 'omp task'
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Keywords: openmp
          Severity: normal
          Priority: P3
         Component: middle-end
          Assignee: unassigned at gcc dot gnu.org
          Reporter: burnus at gcc dot gnu.org
                CC: jakub at gcc dot gnu.org
  Target Milestone: ---

Just to make sure we do not forget it – applies to C/C++/Fortran. The following
is the same in OpenMP 5.0 and 5.1:


  omp task[clause[ [,] clause] ... ] new-line
...
where clause is one of the following:
...
  affinity([aff-modifier:] locator-list)
...

where  aff-modifier  is one of the following:

  iterator(iterators-definition)


The affinity clause is a hint to indicate data affinity of the generated task.
The task is recommended to execute closely to the location of the list items. A
program that relies on the task execution location being determined by this
list may have unspecified behavior.

The list items that appear in the affinity clause may reference iterators
defined by an iterators-definition appearing in the same clause. The list items
that appear in the affinity clause may include array sections.

---v--- C / C++ ---v---
The list items that appear in the affinity clause may use shape-operators.
---^--- C / C++ ---^---

If a list item appears in an affinity clause then data affinity refers to the
original list item.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug middle-end/99666] [OpenMP][5.0] Support 'affinity' clause in 'omp task'
  2021-03-19 11:32 [Bug middle-end/99666] New: [OpenMP][5.0] Support 'affinity' clause in 'omp task' burnus at gcc dot gnu.org
@ 2021-03-19 11:35 ` jakub at gcc dot gnu.org
  2023-07-14 10:05 ` burnus at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: jakub at gcc dot gnu.org @ 2021-03-19 11:35 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99666

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Yeah, but not really sure if we want to do anything with it other than accept
it / verify the restrictions.
The tasking is already overly complicated and the right way is to make it more
scalable (have per thread task queues with work stealing rather than the 3
different queues we currently have with per-team lock).

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug middle-end/99666] [OpenMP][5.0] Support 'affinity' clause in 'omp task'
  2021-03-19 11:32 [Bug middle-end/99666] New: [OpenMP][5.0] Support 'affinity' clause in 'omp task' burnus at gcc dot gnu.org
  2021-03-19 11:35 ` [Bug middle-end/99666] " jakub at gcc dot gnu.org
@ 2023-07-14 10:05 ` burnus at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: burnus at gcc dot gnu.org @ 2023-07-14 10:05 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99666

Tobias Burnus <burnus at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Tobias Burnus <burnus at gcc dot gnu.org> ---
'affinity' including iterator is parsed since r12-1108-g9a5de4d5af1c10 - with a
bunch of follow-up commits fixing various issues.

However, it is ignored after parsing and not used for thread placement.
The latter is tracked as PR110663

Hence, this PR can be closed as FIXED (in/since GCC 12).

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-07-14 10:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-19 11:32 [Bug middle-end/99666] New: [OpenMP][5.0] Support 'affinity' clause in 'omp task' burnus at gcc dot gnu.org
2021-03-19 11:35 ` [Bug middle-end/99666] " jakub at gcc dot gnu.org
2023-07-14 10:05 ` burnus at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).