This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG9c2cfaaada66: [Clang][OpenMP] Allow `f16` literal suffix
when compiling OpenMP target… (authored by tianshilei1992).
Repository:
rG LLVM Github Mo
tianshilei1992 updated this revision to Diff 490375.
tianshilei1992 added a comment.
fix comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142075/new/
https://reviews.llvm.org/D142075
Files:
clang/lib/Lex/LiteralSupport.cpp
clang/test/Ope
jhuber6 accepted this revision.
jhuber6 added a comment.
This revision is now accepted and ready to land.
Seems reasonable, maybe update the comment.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142075/new/
https://reviews.llvm.org/D142075
__
tianshilei1992 created this revision.
tianshilei1992 added reviewers: jdoerfert, jhuber6, yaxunl, rjmccall, tra.
Herald added subscribers: mattd, gchakrabarti, asavonic, guansong.
Herald added a project: All.
tianshilei1992 requested review of this revision.
Herald added subscribers: cfe-commits, s
