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

[Andrey Churbanov via Phabricator via Openmp-commits]

AndreyChurbanov added a comment.

In D65285#1604482 <https://reviews.llvm.org/D65285#1604482>, @AndreyChurbanov wrote:

> 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.
>
> One mode detail - the link I provided is not an "official" location of ittnotify, but just one more usage of it by SEAPI project.  The updates of the ittnotify itself were published sporadically.  And it will get its own stable location on githab soon (the publishing is in process).


Just for the record - the ittnotify API was finally published at user friendly location https://github.com/intel/ittapi (about a week ago; took a bit longer than expected for me).


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65285