IntenseWebs/openvino
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[Gna][coverity] fixes for issue type AUTO_CAUSES_COPY (#17192)

Browse Source 
* [Gna][coverity] fixes for AUTO_CAUSES_COPY CID: 1491505, 1491595, 1502494, 1502500, 1504698, 1504769, 1507058

* update afte review

* adding const specifier to auto where needed
This commit is contained in:
Tomasz Adamowicz
2023-04-26 14:32:54 +02:00
committed by GitHub
parent cfbfa18f34
commit 546581bcce
4 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/plugins/intel_gna/legacy/include/legacy/graph_tools.hpp
View File

@@ -191,8 +191,8 @@ inline void UnorderedDFS(std::unordered_set<CNNLayer*>& visited,
visited.insert(cnnLayer.get());
// visit children
for (auto& od : cnnLayer->outData) {
for (auto nl : getInputTo(od)) {
for (const auto& od : cnnLayer->outData) {
for (const auto& nl : getInputTo(od)) {
layers.push(nl.second);
}
}
@@ -612,7 +612,7 @@ inline CNNNetwork CNNNetCopy(const CNNNetwork& input, const Copier& cp) {
InputsDataMap inputsInfo = input.getInputsInfo();
std::set<DataPtr> insDatas;
for (auto&& info : inputsInfo) {
for (auto secondLayer : getInputTo(info.second->getInputData())) {
for (const auto& secondLayer : getInputTo(info.second->getInputData())) {
auto secondLayerNew = oldToNewLayers[secondLayer.second.get()];
InputInfo::Ptr infoNew = std::make_shared<InputInfo>();
infoNew->setInputData(

4
src/plugins/intel_gna/src/gna_infer_request.cpp
View File

@@ -162,11 +162,11 @@ void GNAInferRequest::CreateInferRequest() {
}
// copy inputs blobs since we need to have them in separate address space to allow simultaneous infer requests
for (auto output : _networkOutputs) {
for (const auto& output : _networkOutputs) {
_outputs[output.first] = plg->GetOutputBlob(output.first, output.second->getTensorDesc().getPrecision());
}
for (auto input : _networkInputs) {
for (const auto& input : _networkInputs) {
_inputs[input.first] = plg->GetInputBlob(input.first, input.second->getTensorDesc().getPrecision());
}
}

2
src/plugins/intel_gna/src/gna_plugin.cpp
View File

@@ -351,7 +351,7 @@ void GNAPlugin::PrePostProcess(InferenceEngine::Blob::Ptr input_blob,
const ov::Shape& input_shape = input_blob->getTensorDesc().getDims();
const ov::Shape& output_shape = output_blob->getTensorDesc().getDims();
for (auto param : model->get_parameters()) {
for (const auto& param : model->get_parameters()) {
param->set_element_type(input_type);
}
model->validate_nodes_and_infer_types();

4
src/plugins/intel_gna/src/optimizer/gna_pass_manager.cpp
View File

@@ -823,7 +823,7 @@ void RemovePermutationsNHWCToNCHWPass::run() {
!getInputTo(pattern_start->outData.front()).empty()) {
auto layer_before_permute = CNNNetPrevLayer(pattern_start);
DataPtr output = nullptr;
for (auto before_output : layer_before_permute->outData) {
for (const auto& before_output : layer_before_permute->outData) {
if (areEqualDatas(pattern_start->input(), before_output)) {
output = before_output;
output->setLayout(getTransposedLayout(output));
@@ -1024,7 +1024,7 @@ void InsertCopyLayerPass::run() {
LayerInfo(l).isSplit()) {
std::vector<FuncChildrenInfo> copy_insertion_tuples;
std::vector<FuncChildrenInfo> delayed_copy_insertion_tuples;
for (auto output : l->outData) {
for (const auto& output : l->outData) {
auto& inputTo = getInputTo(output);
for (auto& childLayer : inputTo) {
std::vector<int> connections = CNNLayerFindInsDataIdxes(output, childLayer.second);