[libcxx-commits] [PATCH] D91379: [libc++] Factor out common logic for calling aligned allocation

[dmajor via Phabricator via libcxx-commits]

dmajor added a comment.

In D91379#2427960 <https://reviews.llvm.org/D91379#2427960>, @mstorsjo wrote:

> I haven't run into this issue - how do you end up getting intrin.h included before the other includes here?

Any case with intrin.h coming first should do it:

  $ cat test.cpp
  #include <intrin.h>
  #include <new>
  
  $ ./clang/bin/x86_64-w64-mingw32-clang++ -c test.cpp 
  In file included from test.cpp:2:
  /home/vm/Desktop/mingwtest/clang/bin/../x86_64-w64-mingw32/include/c++/v1/new:314:12: error: no member named '_aligned_malloc' in the global namespace
    return ::_aligned_malloc(__size, __alignment);
           ~~^
  /home/vm/Desktop/mingwtest/clang/bin/../x86_64-w64-mingw32/include/c++/v1/new:326:5: error: no type named '_aligned_free' in the global namespace
    ::_aligned_free(__ptr);
    ~~^
  2 errors generated.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91379