Details
- Reviewers
aaron.ballman erichkeane shafik cor3ntin
Diff Detail
Event Timeline
clang/lib/AST/Interp/InterpBuiltin.cpp | ||
---|---|---|
88–92 | Allowing char8_t seems wrong to me -- https://godbolt.org/z/zK74hMP7q | |
147–149 | Let's either implement support for them in this patch or exclude the wchar_t changes entirely from the patch? | |
clang/test/AST/Interp/builtin-functions.cpp | ||
225–229 | Why are these in a #if 0 block? |
clang/test/AST/Interp/builtin-functions.cpp | ||
---|---|---|
17 | How about -1 size as well. |
clang/lib/AST/Interp/InterpBuiltin.cpp | ||
---|---|---|
88–92 | Just agreeing with Aaron there. |
There are still unaddressed comment here. It would be nice to complete this before phab shuts down
I have a local patch that implements these builtins by doing a bitcast to a buffer first. There is a comment in ExprConstant.cpp that talks about the same thing. That path probably makes more sense? I'd abandon this review then.
Allowing char8_t seems wrong to me -- https://godbolt.org/z/zK74hMP7q