gribozavr2 added a comment. In D87683#2274883 <https://reviews.llvm.org/D87683#2274883>, @martong wrote:
> In D87683#2274663 <https://reviews.llvm.org/D87683#2274663>, > @baloghadamsoftware wrote: > >> In D87683#2274569 <https://reviews.llvm.org/D87683#2274569>, @njames93 wrote: >> >>> Please fix the test case first, can't call `operator new(unsigned long, >>> void*)` with an argument of type `void*` >>> The other failures the pre merge bot detected can safely be disregarded >> >> Placement new is defined per standard: >> >> void* operator new ( std::size_t count, void* ptr ); >> >> Here, the problem is that `size_t` is `unsigned long` on //Linux// and it >> seems that it is `unsigned long long` on //Windows//. How should I overcome >> this? > > In CSA, to overcome the platform problem, it is common to add a target triple > for the RUN line: > https://github.com/llvm/llvm-project/blob/master/clang/test/Analysis/std-c-library-functions.c#L14 Please make the tests generic if possible: typedef decltype(sizeof(void*)) size_t; Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87683/new/ https://reviews.llvm.org/D87683 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
