Szelethus added inline comments.
================ Comment at: cfe/trunk/test/Analysis/inlining/placement-new-fp-suppression.cpp:16 + +typedef unsigned long uintptr_t; + ---------------- Szelethus wrote: > 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? I seem to have been proven wrong. :) Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62926/new/ https://reviews.llvm.org/D62926 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
