[Openmp-commits] [PATCH] D65285: [OpenMP] Rename last file to cpp and remove LIBOMP_CFLAGS

Mon Jul 29 07:54:13 PDT 2019

AndreyChurbanov added a comment.

In D65285#1604409 <https://reviews.llvm.org/D65285#1604409>, @Hahnfeld wrote:

> In D65285#1602406 <https://reviews.llvm.org/D65285#1602406>, @AndreyChurbanov wrote:
>
> > I will contact ittnotify owners just in case. So that we might have lesser burden with possible future updates of this third-party code.
>
>
> Thanks. Is that open-source or internal to Intel?

It is open-sourced many years ago, now under BSD license; IINM, it can be found at https://github.com/intel/IntelSEAPI/tree/master/ittnotify (though we don't update it regularly without real need);
and it is also part of Intel Tools products (e.g. Amplifier, etc.).

They already agreed to upstream the last change (removal of const).  Not sure it will be possible to upstream .c --> .cpp, because they depend on many other customers. But I think this is minor issue for us.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65285