[PATCH] D117091: [Clang] Add attributes alloc_size and alloc_align to mm_malloc

[Dávid Bolvanský via Phabricator via cfe-commits]

xbolva00 added a comment.

In D117091#3272589 <https://reviews.llvm.org/D117091#3272589>, @aaron.ballman wrote:

> There's a testing issue on Windows:
>
>   ******************** TEST 'Clang :: Headers/mm_malloc.c' FAILED ********************
>   Script:
>   --
>   : 'RUN: at line 1';   c:\ws\w2\llvm-project\premerge-checks\build\bin\clang.exe -emit-llvm -std=c11 -x c C:\ws\w2\llvm-project\premerge-checks\clang\test\Headers\mm_malloc.c -O1 --target=x86_64-linux-gnu -S -o - | c:\ws\w2\llvm-project\premerge-checks\build\bin\filecheck.exe C:\ws\w2\llvm-project\premerge-checks\clang\test\Headers\mm_malloc.c
>   --
>   Exit Code: 2
>   
>   Command Output (stdout):
>   --
>   $ ":" "RUN: at line 1"
>   $ "c:\ws\w2\llvm-project\premerge-checks\build\bin\clang.exe" "-emit-llvm" "-std=c11" "-x" "c" "C:\ws\w2\llvm-project\premerge-checks\clang\test\Headers\mm_malloc.c" "-O1" "--target=x86_64-linux-gnu" "-S" "-o" "-"
>   # command stderr:
>   In file included from C:\ws\w2\llvm-project\premerge-checks\clang\test\Headers\mm_malloc.c:2:
>   
>   c:\ws\w2\llvm-project\premerge-checks\build\lib\clang\14.0.0\include\mm_malloc.h:13:10: fatal error: 'stdlib.h' file not found
>   
>   #include <stdlib.h>
>   
>            ^~~~~~~~~~
>   
>   1 error generated.
>   
>   
>   error: command failed with exit status: 1
>   $ "c:\ws\w2\llvm-project\premerge-checks\build\bin\filecheck.exe" "C:\ws\w2\llvm-project\premerge-checks\clang\test\Headers\mm_malloc.c"
>   # command stderr:
>   FileCheck error: '<stdin>' is empty.
>   
>   FileCheck command line:  c:\ws\w2\llvm-project\premerge-checks\build\bin\filecheck.exe C:\ws\w2\llvm-project\premerge-checks\clang\test\Headers\mm_malloc.c
>   
>   
>   error: command failed with exit status: 2
>
> This one is neat because `stdlib.h` is not a header provided by the compiler but is instead provided by whatever CRT happens to be used. Normally, we require those headers to be mocked, but the goal here is to test the header the compiler does provide.

Thanks for checking, I have no suitable Windows device to run patch on it.


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

https://reviews.llvm.org/D117091