tbaeder added inline comments.

================
Comment at: clang/test/AST/Interp/builtin-functions.cpp:53
+  constexpr char f[] = {'0', 'x', 'A', 'E', '\0'};
+  constexpr double NaN6 = __builtin_nan(f); // ref-error {{must be initialized 
by a constant expression}}
+
----------------
aaron.ballman wrote:
> This should be accepted, shouldn't it? The payload is a valid integer 
> constant (as a string).
So answer both your questions: Yes, but the current interpreter has a 
`isa<StringLiteral>()` check and rejects everything that isn't a string literal.


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

https://reviews.llvm.org/D155356

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to