message: [OpenMP][libomptarget] Set the frame pointer then test empty slot condition
  Summary: NFC - just fixing a bug: the empty slot test was before the re-setting of the Stack pointer.
  Reviewers: ABataev, caomhin, Hahnfeld
  Subscribers: guansong, openmp-commits
diff: @@ -477,13 +477,13 @@ EXTERN void __kmpc_data_sharing_pop_stack(void *FrameStart) {
  // Pointer to next available stack.
  void *&StackP = DataSharingState.StackPtr[WID];
  + // Pop the frame.
  + StackP = FrameStart;
  +
  // If the current slot is empty, we need to free the slot after the
  // pop.
  bool SlotEmpty = (StackP == &SlotP->Data[0]);
  - // Pop the frame.
  - StackP = FrameStart;
  -
  if (SlotEmpty && SlotP->Prev) {
  // Before removing the slot we need to reset StackP.
  StackP = SlotP->PrevSlotStackPtr;
  