commit b86bf83c281cbc1c42de3dfca6ae2555094abeae
author Johannes Doerfert <johannes@jdoerfert.de> Sat Feb 15 18:07:42 2020 -0600
committer Johannes Doerfert <johannes@jdoerfert.de> Sat Feb 15 18:09:54 2020 -0600
tree 9dc241a2e5ede1ace7f588220484544b02116a40
parent 82b47b2978405f802a33b00d046e6f18ef6a47be

[FIX] Remove pointer in attribute to eliminate leaks (see D71830)

clang/lib/Parse/ParseOpenMP.cpp

diff --git a/clang/lib/Parse/ParseOpenMP.cpp b/clang/lib/Parse/ParseOpenMP.cpp
index e1bcbdb..8c822ec 100644
--- a/clang/lib/Parse/ParseOpenMP.cpp
+++ b/clang/lib/Parse/ParseOpenMP.cpp
@@ -1365,26 +1365,24 @@
   }
 
   // Parse inner context selectors.
-  OMPTraitInfo *TI = new OMPTraitInfo();
-  parseOMPContextSelectors(Loc, *TI);
+  OMPTraitInfo TI;
+  parseOMPContextSelectors(Loc, TI);
 
   // Parse ')'
   (void)T.consumeClose();
 
   Optional<std::pair<FunctionDecl *, Expr *>> DeclVarData =
       Actions.checkOpenMPDeclareVariantFunction(
-          Ptr, AssociatedFunction.get(), *TI,
+          Ptr, AssociatedFunction.get(), TI,
           SourceRange(Loc, Tok.getLocation()));
 
   // Skip last tokens.
   while (Tok.isNot(tok::annot_pragma_openmp_end))
     ConsumeAnyToken();
-  if (DeclVarData.hasValue() && !TI->Sets.empty())
+  if (DeclVarData.hasValue() && !TI.Sets.empty())
     Actions.ActOnOpenMPDeclareVariantDirective(
         DeclVarData.getValue().first, DeclVarData.getValue().second, TI,
         SourceRange(Loc, Tok.getLocation()));
-  else
-    delete TI;
 
   // Skip the last annot_pragma_openmp_end.
   (void)ConsumeAnnotationToken();