Displaying 3 results from an estimated 3 matches for "r259255".
Did you mean:
r259252
2016 Jan 30
1
[llvm] r259255 - Need #include <cstdint> for uint64_t
On Fri, Jan 29, 2016 at 6:27 PM, Matthias Braun via llvm-commits <
llvm-commits at lists.llvm.org> wrote:
> The point I was trying to make here is that all things that DataTypes.h
> provides:
> intXXX_t, uintXX_t, PRIdXX, INT64_MAX, HUGE_VAL, ssize_t ...
> are all part of the C++11 standard and can be found in <cstdint>, <cmath>
> (except for ssize_t) so I
2016 Jan 30
2
[llvm] r259255 - Need #include <cstdint> for uint64_t
Oh I did not know about DataTypes.h. However given that there are already two other files in llvm (in the AMDGPU and Hexagon targets) that #include <cstdint> since 2014, maybe we can loosen the requirement for DataTypes.h nowadays as #include <cstdint> seems to be working everywhere?
- Matthias
> On Jan 29, 2016, at 3:06 PM, Craig Topper <craig.topper at gmail.com> wrote:
2016 Jan 30
2
[llvm] r259255 - Need #include <cstdint> for uint64_t
DataTypes.h is transitively included through a lot of very common includes
already. So its entirely possible AMDGPU and Hexagon are really including
it.
On Fri, Jan 29, 2016 at 5:02 PM, mats petersson <mats at planetcatfish.com>
wrote:
> Sounds to me like including `DataTypes.h` would be the right choice, since
> it will include `stdint.h` if it exists - and error if it doesn't,