Re: r318601 - [OpenMP] Show error if VLAs are not supported

2017-11-20 Thread Jonas Hahnfeld via cfe-commits
I'm sorry, I didn't get a notification for that failure. I'll reply to the Phabricator. Am 2017-11-20 19:37, schrieb Artem Belevich: Proposed fix: https://reviews.llvm.org/D40275 On Mon, Nov 20, 2017 at 4:13 PM, Artem Belevich wrote: This change breaks CUDA as clang now

Re: r318601 - [OpenMP] Show error if VLAs are not supported

2017-11-20 Thread Artem Belevich via cfe-commits
Proposed fix: https://reviews.llvm.org/D40275 On Mon, Nov 20, 2017 at 4:13 PM, Artem Belevich wrote: > This change breaks CUDA as clang now reports an error during device-side > compilation when VLA is used in the *host-side* code. >

Re: r318601 - [OpenMP] Show error if VLAs are not supported

2017-11-20 Thread Artem Belevich via cfe-commits
This change breaks CUDA as clang now reports an error during device-side compilation when VLA is used in the *host-side* code. http://lab.llvm.org:8011/builders/clang-cuda-build/builds/15591/steps/ninja%20build%20simple%20CUDA%20tests/logs/stdio E.g. I would expect this code to compile

r318601 - [OpenMP] Show error if VLAs are not supported

2017-11-18 Thread Jonas Hahnfeld via cfe-commits
Author: hahnfeld Date: Sat Nov 18 13:00:46 2017 New Revision: 318601 URL: http://llvm.org/viewvc/llvm-project?rev=318601=rev Log: [OpenMP] Show error if VLAs are not supported Some target devices (e.g. Nvidia GPUs) don't support dynamic stack allocation and hence no VLAs. Print errors with