[PATCH] D62926: [analyzer] ReturnVisitor: Bypass everything to see inlined calls

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 16 10:32:35 PDT 2019


Szelethus added inline comments.


================
Comment at: cfe/trunk/test/Analysis/inlining/placement-new-fp-suppression.cpp:16
+
+typedef unsigned long uintptr_t;
+
----------------
hintonda wrote:
> I don't believe this is really ever portable, but definitely not on 64 bit Windows where a long is 32 bits.
> 
> Can't you just `#include <stdint.h>` instead?
In the test, I'm pretty sure that he can't. 

I grepped `test/Analysis`, and found that `malloc.cpp` does the following:

```
typedef unsigned __INTPTR_TYPE__ uintptr_t
```

Maybe that's worth a shot?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62926





More information about the cfe-commits mailing list