[PATCH] omp-low.c split

Alexander Monakov amonakov@ispras.ru
Tue Dec 13 12:58:00 GMT 2016


On Tue, 13 Dec 2016, Martin Jambor wrote:
> I have bootstrapped the two patches on aarch64-linux and bootstrapped
> and tested them on x86_64-linux.  What do you think?

Sorry for my 'false alarm' about cp/parser.c conflict in the previous mail -- I
thought I was applying your patch to trunk, but now I see my tree was outdated.

In the new patch there are 7 whitespace issues that git complains about:

zcat 0001-Split-omp-low-into-multiple-files.patch.gz |
  git apply --check --whitespace=error-all -

<stdin>:2779: space before tab in indent.
   	# BLOCK 2 (PAR_ENTRY_BB)
<stdin>:5391: space before tab in indent.
			       	true, GSI_SAME_STMT);
<stdin>:8174: space before tab in indent.
  	     after a stmt, not before.  */
<stdin>:9105: space before tab in indent.
		 		  GOMP_atomic_start ();
<stdin>:9106: space before tab in indent.
		 		  *addr = rhs;
<stdin>:9107: space before tab in indent.
		 		  GOMP_atomic_end ();
<stdin>:10327: space before tab in indent.
       	 region.  */
error: 7 lines add whitespace errors.


A couple of typos in the Changelog:

> 	(is_combined_parallel): kMoved to omp-expand.c.

s/k//

> 	* config/nvptx/nvptx.c: Include omp-generic.c.

omp-general.h :)

Alexander



More information about the Gcc-patches mailing list