[Openmp-commits] [PATCH] D108574: [OpenMP] libomp: runtime part of omp_all_memory task dependence implementation.

Fri Oct 22 08:35:05 PDT 2021

protze.joachim added a comment.

Writing up the OMPT interface for `omp_all_memory`, I realized that treating `omp_all_memory` individually might not be the best choice, considering that the spec talks about "Reserved Locators".

How final is this compiler interface?

> Library recognizes the new type via either
> (dependence_address == NULL && dependence_flag == 0x80 (set highest bit of 1-byte flag))
> or
> (dependence_address == SIZE_MAX).

My suggestion would be to define `dependence_flag == 0x80` indicating the presence of a //reserved locator//, but still use a specific address value to indicate the concrete locator. I guess for the alternative interface with `SIZE_MAX`, we could use `SIZE_MAX-1`... for upcoming reserved locators.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108574/new/

https://reviews.llvm.org/D108574