[PATCH] Integration of parallel standard algorithms for c++17

Jakub Jelinek jakub@redhat.com
Mon Feb 4 11:30:00 GMT 2019


On Thu, Jan 31, 2019 at 09:08:52PM -0800, Thomas Rodgers wrote:
> Update C++17 parallel algorithms to LLVM/MIT licensed upstream sources

Just ChangeLog formatting nits below:
> 
> 	* include/Makefile.am:update from upstream
> 		remove ${pstl_srcdir}/iterators.h
> 		add ${pstl_srcdir}/memory_impl.h
> 

Generally, the format is TAB * SPACE filename optional function name etc. in
parens COLON SPACE What has changed PERIOD, where what has changed starts
with a capital letter.  Your entries miss the capital letters, spaces and
periods.  include/Makefile.am has GCC as its own upstream, so Update from
upstream. makes no sense.  You've changed something in there, so usually one
writes what has changed:
	* include/Makefile.am (std_headers): Add ${std_srcdir}/execution.
	(pstl_srcdir, pstl_builddir, pstl_headers): New variables.
...
etc.
	* include/Makefile.in: Regenerated.

> 	* include/bits/c++config:update from upstream

Similarly, c++config doesn't have some other upstream, so one needs to say
what has changed.  Try contrib/mklog to get a template and just tweak it
when it doesn't handle something right (it is written mostly for C/C++
sources, not Makefiles etc.).

> 	* include/pstl/algorithm_fwd.h:update from upstreamgg

etc., these are newly added, so one usually writes ...: New file.

	Jakub



More information about the Libstdc++ mailing list