From 5fea6eecde8dde2e122ad29217c3fff33e89ded2 Mon Sep 17 00:00:00 2001 From: Frederic Bastien Date: Fri, 12 Jul 2019 06:20:56 -0700 Subject: [PATCH] Update comment --- tensorflow/compiler/xla/service/gpu/ir_emitter_unnested.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tensorflow/compiler/xla/service/gpu/ir_emitter_unnested.h b/tensorflow/compiler/xla/service/gpu/ir_emitter_unnested.h index a00b80b07e8..8308e97ec99 100644 --- a/tensorflow/compiler/xla/service/gpu/ir_emitter_unnested.h +++ b/tensorflow/compiler/xla/service/gpu/ir_emitter_unnested.h @@ -228,8 +228,10 @@ class IrEmitterUnnested : public IrEmitter, // `scatter_indices_gen`, updates from`updates_gen`. The output buffer is // expected to have the operand values in it already. If use_atomic // is true, we will use an atomic update. Using false for use_atomic - // is safe only when it is guaranteed that there is no duplicate + // is safe only when it is guaranteed that there are no duplicate // indices. + // When using use_atomi=false, it is the caller responsibility to + // ensure there is overlap. Status EmitScatter(Thunk* thunk, HloInstruction* scatter, const llvm_ir::ElementGenerator& scatter_indices_gen, const llvm_ir::ElementGenerator& updates_gen,