Revert deallocation changes + add missing header

I was mistaken about the deallocation. Firstly, all of the other private
copies on threads that don't execute the last iteration need to be
freed. Secondly, the assign copies the data not the pointer.

Author: tblah

flang/lib/Lower/OpenMP/DataSharingProcessor.cpp

@@ -17,6 +17,7 @@
 #include "flang/Lower/SymbolMap.h"
 #include "flang/Optimizer/Builder/HLFIRTools.h"
 #include "flang/Optimizer/Builder/Todo.h"
+#include "flang/Optimizer/HLFIR/HLFIROps.h"
 #include "flang/Semantics/tools.h"
 
 namespace Fortran {
@@ -79,8 +80,7 @@ void DataSharingProcessor::processStep2(mlir::Operation *op, bool isLoop) {
 
 void DataSharingProcessor::insertDeallocs() {
   for (const semantics::Symbol *sym : allPrivatizedSymbols)
-    if (semantics::IsAllocatable(sym->GetUltimate()) &&
-        !sym->test(semantics::Symbol::Flag::OmpLastPrivate)) {
+    if (semantics::IsAllocatable(sym->GetUltimate())) {
       if (!useDelayedPrivatization) {
         converter.createHostAssociateVarCloneDealloc(*sym);
         continue;

flang/test/Lower/OpenMP/lastprivate-allocatable.f90

@@ -36,9 +36,3 @@ program lastprivate_allocatable
 ! CHECK-NEXT:            }
 ! CHECK-NEXT:            omp.yield
 ! CHECK-NEXT:          }
-! CHECK-NEXT:          omp.terminator
-! CHECK-NEXT:        }
-!                    [no deallocation here]
-! CHECK-NEXT:        omp.terminator
-! CHECK-NEXT:      }
-