[PATCH] D53530: Fix (and improve) the support for C99 variable length array types

Stella Stamenova via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 7 14:00:07 PST 2018


stella.stamenova added a comment.

In https://reviews.llvm.org/D53530#1290680, @aprantl wrote:

> In https://reviews.llvm.org/D53530#1290664, @stella.stamenova wrote:
>
> > TestVla fails on Windows: http://lab.llvm.org:8014/builders/lldb-x64-windows-ninja/builds/1129/steps/test/logs/stdio
> >
> >   AssertionError: False is not True : 'frame var vla' returns expected result, got '(int []) vla = {}'
> >
> >
> > I will have time to look in more detail later this week, but if you have any ideas in the mean time, that would be great.
>
>
> Is your bot using DWARF or CodeView as a debug info format? Because this test definitely requires DWARF.


I was under the impression that each test can control how to build the necessary payloads. Is that not the case?


Repository:
  rL LLVM

https://reviews.llvm.org/D53530





More information about the llvm-commits mailing list