[Openmp-commits] [PATCH] D55496: [OpenMP] RFC: fixes for LIBOMP_OMP_VERSION=4.5/4
Roman Lebedev via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Tue Dec 11 10:14:40 PST 2018
lebedev.ri added a comment.
In D55496#1327109 <https://reviews.llvm.org/D55496#1327109>, @AndreyChurbanov wrote:
> In D55496#1324768 <https://reviews.llvm.org/D55496#1324768>, @lebedev.ri wrote:
>
> > In D55496#1324745 <https://reviews.llvm.org/D55496#1324745>, @Hahnfeld wrote:
> >
> > > II think it would be best to just remove all the `#ifdef`s and code that is not needed anymore.
> >
> >
> > Sure, i could do that instead, if that is the final decision.
> > I just don't want it to stay in the current sudo-broken state.
>
>
> I don't mind removing the OpenMP versioning from sources, but this is much bigger change which will be harder to review.
... and the one that would need an RFC i would imagine.
> Feel free to commit this for now, or work on the removing of all OpenMP versioning.
Ok, thank you for the review.
Repository:
rOMP OpenMP
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55496/new/
https://reviews.llvm.org/D55496
More information about the Openmp-commits
mailing list