[Openmp-commits] [PATCH] D106627: [OpenMP] Add environment variables to change stack / heap size in the CUDA plugin

[Joseph Huber via Phabricator via Openmp-commits]

jhuber6 added a comment.

In D106627#2899398 <https://reviews.llvm.org/D106627#2899398>, @pengfei wrote:

> In D106627#2899368 <https://reviews.llvm.org/D106627#2899368>, @abhinavgaba wrote:
>
>> In D106627#2899301 <https://reviews.llvm.org/D106627#2899301>, @pengfei wrote:
>>
>>> There're three fails in my local check-all. Is this expected?
>>>
>>>   Failed Tests (3):
>>>     libomptarget :: x86_64-pc-linux-gnu :: offloading/memory_manager.cpp
>>>     libomptarget :: x86_64-pc-linux-gnu :: offloading/parallel_offloading_map.cpp
>>>     libomptarget :: x86_64-pc-linux-gnu :: offloading/taskloop_offload_nowait.cpp
>>
>> Pengfei, these fails are from a different part of code, that should not be affected by this change.
>>
>> I see these fails in my environment (RHEL 8.2, gcc 8.3.1) for commit `4a76bd0e` (previous buildable commit) as well.
>
> I see. Then I commited it rGf7c92995c0e1f95 <https://reviews.llvm.org/rGf7c92995c0e1f95ef5640a0df325e05d12de4658>.

Thanks for fixing it, I forgot about building without the SDK.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106627/new/

https://reviews.llvm.org/D106627