Parallel STL version updated

[ldionne]

Summary

PSTL_VERSION 203 - > PSTL_VERSION 205

[ldionne]

What determines the version? This doesn't seem to be tied to any LLVM release, is it?

[MikeDvorskiy]

Yes,... I doubted that is relevant to LLVM release.. and so as the some other macros form pstl_config.h
But as far as pstl_config.h is, PSTL_VERSION may be useful, at least to differ major changes, I guess...

[ldionne]

We should talk about it, but we could consider tying versions of the PSTL to versions of LLVM. Or maybe some other scheme makes more sense. But I'm sure we can come up with something more useful than just a random number!

[MikeDvorskiy]

Yes, agree.

[Eugene.Zelenko]

include/pstl/internal/pstl_config.h

1

I think -*- C++ -*- should be merged into next line. See LLVM and Clang headers as example.

[dexonsmith]

include/pstl/internal/pstl_config.h

1

That's correct, but it's also unrelated to this commit. I would suggest fixing it separately.

[ldionne]

This review isn't relevant anymore, the _PSTL_VERSION is now tied to LLVM's version.

[ldionne]

Abandoned to clear up the review queue.

include/pstl/internal/pstl_config.h

 // -*- C++ -*-
 //===-- pstl_config.h -----------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 #ifndef __PSTL_config_H
 #define __PSTL_config_H
 
-#define PSTL_VERSION 203
+#define PSTL_VERSION 205
 #define PSTL_VERSION_MAJOR (PSTL_VERSION / 100)
 #define PSTL_VERSION_MINOR (PSTL_VERSION - PSTL_VERSION_MAJOR * 100)
 
 // Check the user-defined macro for parallel policies
 #if defined(PSTL_USE_PARALLEL_POLICIES)
 #undef __PSTL_USE_PAR_POLICIES
 #define __PSTL_USE_PAR_POLICIES PSTL_USE_PARALLEL_POLICIES
 // Check the internal macro for parallel policies