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...


Repository:
  rL LLVM

https://reviews.llvm.org/D29660



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to