tbaeder added a comment.
Meh, is `__fp16` something I need to check support for?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146436/new/
https://reviews.llvm.org/D146436
___
cfe-commits mailing list
cf
This revision was automatically updated to reflect the committed changes.
Closed by commit rG0691bcb18024: [clang][Interp][NFC] Add tests for __fp16
(authored by tbaeder).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146436/new/
https://reviews.llv
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146436/new/
https://reviews.llvm.org/D146436
___
cfe-commits mailing list
tbaeder added inline comments.
Comment at: clang/test/AST/Interp/floats.cpp:110-111
+
+ static_assert(~2.0f == 3, ""); // ref-error {{invalid argument type 'float'
to unary expression}} \
+ // expected-error {{invalid argument type
'float' to un
tbaeder updated this revision to Diff 506701.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146436/new/
https://reviews.llvm.org/D146436
Files:
clang/test/AST/Interp/floats.cpp
Index: clang/test/AST/Interp/floats.cpp
===
aaron.ballman added inline comments.
Comment at: clang/test/AST/Interp/floats.cpp:110-111
+
+ static_assert(~2.0f == 3, ""); // ref-error {{invalid argument type 'float'
to unary expression}} \
+ // expected-error {{invalid argument type
'float'
tbaeder created this revision.
tbaeder added reviewers: aaron.ballman, erichkeane, tahonermann, shafik.
Herald added a project: All.
tbaeder requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This should already work just fine since our `Float
