This is an archive of the discontinued LLVM Phabricator instance.

[OPENMP][NVPTX]Fix __kmpc_flush to flush the memory per system, not per block.
ClosedPublic

Authored by ABataev on Dec 6 2018, 7:16 AM.

Reviewers

gtbercea
kkwli0

Commits

Summary

According to the standard, after memory flushing the changes in the
memory must be visible to all the threads in all teams. Patch fixes
this.

Repository

rOMP OpenMP

Build Status

Buildable 25772
Build 25771: arc lint + arc unit

This revision is now accepted and ready to land.Dec 6 2018, 7:20 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

libomptarget/

deviceRTLs/

nvptx/

src/

2 lines

	Show First 20 Lines • Show All 126 Lines • ▼ Show 20 Lines
	}			}

	////////////////////////////////////////////////////////////////////////////////			////////////////////////////////////////////////////////////////////////////////
	// Flush			// Flush
	////////////////////////////////////////////////////////////////////////////////			////////////////////////////////////////////////////////////////////////////////

	EXTERN void __kmpc_flush(kmp_Ident *loc) {			EXTERN void __kmpc_flush(kmp_Ident *loc) {
	PRINT0(LD_IO, "call kmpc_flush\n");			PRINT0(LD_IO, "call kmpc_flush\n");
	__threadfence_block();			__threadfence_system();
	}			}

	////////////////////////////////////////////////////////////////////////////////			////////////////////////////////////////////////////////////////////////////////
	// Vote			// Vote
	////////////////////////////////////////////////////////////////////////////////			////////////////////////////////////////////////////////////////////////////////

	EXTERN int32_t __kmpc_warp_active_thread_mask() {			EXTERN int32_t __kmpc_warp_active_thread_mask() {
	PRINT0(LD_IO, "call __kmpc_warp_active_thread_mask\n");			PRINT0(LD_IO, "call __kmpc_warp_active_thread_mask\n");
	return __ACTIVEMASK();			return __ACTIVEMASK();
	}			}