This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGb395e91f2ef4: [clang][Interp] Implement __builtin_inf() etc.
(authored by tbaeder).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
cor3ntin accepted this revision.
cor3ntin added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/lib/AST/Interp/InterpBuiltin.cpp:119
+static bool interp__builtin_inf(InterpState &S, CodePtr OpPC,
+const InterpF
tbaeder added inline comments.
Comment at: clang/lib/AST/Interp/InterpBuiltin.cpp:119
+static bool interp__builtin_inf(InterpState &S, CodePtr OpPC,
+const InterpFrame *Frame, const Function *F) {
cor3ntin wrote:
> why does that
cor3ntin added inline comments.
Comment at: clang/lib/AST/Interp/InterpBuiltin.cpp:119
+static bool interp__builtin_inf(InterpState &S, CodePtr OpPC,
+const InterpFrame *Frame, const Function *F) {
why does that return a bool? yo
tbaeder updated this revision to Diff 540774.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155367/new/
https://reviews.llvm.org/D155367
Files:
clang/lib/AST/Interp/InterpBuiltin.cpp
Index: clang/lib/AST/Interp/InterpBuiltin.cpp
===
tbaeder added a comment.
Tests will come in a later commit.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155367/new/
https://reviews.llvm.org/D155367
___
cfe-commits mailing list
cfe-commits@lists.llvm.
tbaeder created this revision.
tbaeder added reviewers: aaron.ballman, erichkeane, shafik, cor3ntin.
Herald added a project: All.
tbaeder requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Repository:
rG LLVM Github Monorepo
https://reviews
