sbc100 added a comment.
Please ignore this change, I had thought we wanted size_t to match either
int32_t or int64_t, but it turns out that isn't a requirement and that code
that expects this to be true needs fixing.
Repository:
rC Clang
https://reviews.llvm.org/D50477
joerg added a comment.
I don't understand the desire for this logic. Why can't wasm override the rest
of the types if it wants to have something special?
Repository:
rC Clang
https://reviews.llvm.org/D50477
___
cfe-commits mailing list
sbc100 updated this revision to Diff 159794.
sbc100 added a comment.
remove debugging
Repository:
rC Clang
https://reviews.llvm.org/D50477
Files:
lib/Frontend/InitPreprocessor.cpp
Index: lib/Frontend/InitPreprocessor.cpp
sbc100 created this revision.
Herald added subscribers: cfe-commits, sunfish, aheejin.
We recently changes `size_t` on wasm from `int` to `long` which
had the effect of making the type of size_t (long) not match either
int32_t or int64_t.
This solution is not very elegant but fixes the build