experiments/STT-tensorflow
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix bug on referencing invalid reference to a tensor.

Browse Source 
Adding tensors to the TfLiteContext will give TfLiteContext.tensors a new address, so the old references should be updated with new one.

PiperOrigin-RevId: 319188033
Change-Id: I1538d6260236f7cf5a710621d6af330a8639f443
This commit is contained in:
Taehee Jeong 2020-07-01 02:26:51 -07:00 committed by TensorFlower Gardener
parent 6fc54250a5
commit fc49cbb2ad
3 changed files with 8 additions and 3 deletions
tensorflow/lite/delegates
gpu/common
object_reader.cc
utils.ccutils.h

6
tensorflow/lite/delegates/gpu/common/object_reader.cc
View File

@ -37,7 +37,7 @@ absl::Status ObjectReader::ReadNonConstantTensor(
}
if (tensor_to_value->find(tensor_idx) == tensor_to_value->end()) {
const TfLiteTensor& tflite_tensor = context->tensors[tensor_idx];
TfLiteTensor& tflite_tensor = context->tensors[tensor_idx];
if (tflite::IsConstantTensor(&tflite_tensor)) {
return absl::InvalidArgumentError(absl::StrCat(
"ReadNonConstantTensor: value is a constant tensor: ", tensor_idx));
@ -58,6 +58,7 @@ absl::Status ObjectReader::ReadNonConstantTensor(
&fp_tensor_index) != kTfLiteOk) {
return absl::InternalError("Could not add new tensor to graph");
}
// Remember this tensor for later.
(*quant_conversion_map)[fp_tensor_index] = tensor_idx;
(*quant_conversion_map)[tensor_idx] = fp_tensor_index;
@ -67,6 +68,9 @@ absl::Status ObjectReader::ReadNonConstantTensor(
ConvertTfLiteTensorToTensorRef(*fp_tflite_tensor, &value->tensor));
value->tensor.ref = fp_tensor_index;
value->quant_params.emplace();
// tflite_tensor from the outer scope is invalidated due to calling
// CreateNewTensorWithDifferentType
tflite_tensor = context->tensors[tensor_idx];
RETURN_IF_ERROR(
PopulateQuantParams(tflite_tensor, &value->quant_params.value()));
(*tensor_to_value)[fp_tensor_index] = value;

2
tensorflow/lite/delegates/utils.cc
View File

@ -29,8 +29,8 @@ TfLiteStatus CreateNewTensorWithDifferentType(TfLiteContext* context,
TfLiteType new_type,
TfLiteTensor** new_tensor,
int* new_tensor_index) {
const TfLiteTensor& original_tensor = context->tensors[original_tensor_index];
TF_LITE_ENSURE_STATUS(context->AddTensors(context, 1, new_tensor_index));
const TfLiteTensor& original_tensor = context->tensors[original_tensor_index];
*new_tensor = &context->tensors[*new_tensor_index];
(*new_tensor)->type = new_type;
(*new_tensor)->allocation_type = kTfLiteArenaRw;

3
tensorflow/lite/delegates/utils.h
View File

@ -33,7 +33,8 @@ namespace tflite {
namespace delegates {
// Creates a new Read/Write tensor having the same shape as the original, but
// with a different type.
// with a different type. Note that this might void existing references to
// tensors.
TfLiteStatus CreateNewTensorWithDifferentType(TfLiteContext* context,
const int original_tensor_index,
TfLiteType new_type,