[PATCH] D38259: [OpenMP] Fix translation of target args

2017-09-27 Thread Jonas Hahnfeld via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314330: [OpenMP] Fix translation of target args (authored by Hahnfeld). Changed prior to commit: https://reviews.llvm.org/D38259?vs=116610=116846#toc Repository: rL LLVM

[PATCH] D38259: [OpenMP] Fix translation of target args

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/D38259 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D38259: [OpenMP] Fix translation of target args

2017-09-25 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld created this revision. ToolChain::TranslateArgs() returns nullptr if no changes are performed. This would currently mean that OpenMPArgs are lost. Patch fixes this by falling back to simply using OpenMPArgs in that case. https://reviews.llvm.org/D38259 Files: