[PATCH] D29660: [OpenMP] Add flag for overwriting default PTX version for OpenMP targets
Jonas Hahnfeld via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 14 00:53:15 PDT 2017
Hahnfeld added a comment.
In https://reviews.llvm.org/D29660#839736, @alekseyshl wrote:
> In https://reviews.llvm.org/D29660#839728, @alekseyshl wrote:
> > Bad news, the bot is still red: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/7114
> Disabled openmp-offload.c on Linux again: https://reviews.llvm.org/rL310772
I think I've found the memory leak with `cmake -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_ASSERTIONS=ON -DLLVM_USE_SANITIZER=Address -DLLVM_ENABLE_LIBCXX=ON`.
I committed a fix, let's see if the bot likes `openmp-offload.c` in https://reviews.llvm.org/rL310817. There is another leak for `openmp-offload-gpu.c` which I will write about in https://reviews.llvm.org/D34784.
Side note: We might want to get rid of `2>&1` in all the tests so that sanitizer errors get through! That would have sped up the search...
More information about the cfe-commits