[PATCH] D38257: [OpenMP] Fix memory leak when translating arguments

2017-09-27 Thread Jonas Hahnfeld via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314328: [OpenMP] Fix memory leak when translating arguments (authored by Hahnfeld). Changed prior to commit: https://reviews.llvm.org/D38257?vs=116607&id=116844#toc Repository: rL LLVM https://revie

[PATCH] D38257: [OpenMP] Fix memory leak when translating arguments

2017-09-27 Thread Gheorghe-Teodor Bercea via Phabricator via cfe-commits
gtbercea accepted this revision. gtbercea added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D38257 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/c

[PATCH] D38257: [OpenMP] Fix memory leak when translating arguments

2017-09-25 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld created this revision. Parsing the argument after -Xopenmp-target allocates memory that needs to be freed. Associate it with the final DerivedArgList after we know which one will be used. https://reviews.llvm.org/D38257 Files: include/clang/Driver/ToolChain.h lib/Driver/Compilation