[OPENMP]Increment iterator when the loop is continued.

[ABataev]

Summary

Missed operation of the incrementing iterator when required just to
continue execution.

[gtbercea]

LG

[Hahnfeld]

(for reference: I missed this in D51623)

@ABataev can you extend test/mapping/pr38704.c with a map that would have triggered the infinite loop?

[ABataev]

In D51937#1247422, @Hahnfeld wrote:

(for reference: I missed this in D51623)

@ABataev can you extend test/mapping/pr38704.c with a map that would have triggered the infinite loop?

Yes, will try to do it.

[ABataev]

Extended the test in r343344

openmp/trunk/libomptarget/src/omptarget.cpp

if ((arg_types[i] & OMP_TGT_MAPTYPE_FROM) || DelEntry) {
       uintptr_t ub = (uintptr_t) HstPtrBegin + data_size;
       Device.ShadowMtx.lock();
       for (ShadowPtrListTy::iterator it = Device.ShadowPtrMap.begin();
            it != Device.ShadowPtrMap.end();) {
         void **ShadowHstPtrAddr = (void**) it->first;
 
         // An STL map is sorted on its keys; use this property
         // to quickly determine when to break out of the loop.
-        if ((uintptr_t) ShadowHstPtrAddr < lb)
+        if ((uintptr_t) ShadowHstPtrAddr < lb) {
+          ++it;
           continue;
+        }
         if ((uintptr_t) ShadowHstPtrAddr >= ub)
           break;
 
         // If we copied the struct to the host, we need to restore the pointer.
         if (arg_types[i] & OMP_TGT_MAPTYPE_FROM) {
           DP("Restoring original host pointer value " DPxMOD " for host "
               "pointer " DPxMOD "\n", DPxPTR(it->second.HstPtrVal),
               DPxPTR(ShadowHstPtrAddr));