This revision was automatically updated to reflect the committed changes.
Closed by commit rL363493: [clang] Add storage for APValue in ConstantExpr
(authored by Tyker, committed by ).
Herald added a subscriber: kristina.
Changed prior to commit:
https://reviews.llvm.org/D62399?vs=203723&id=204
Tyker updated this revision to Diff 203723.
Tyker marked 4 inline comments as done.
Tyker edited the summary of this revision.
Tyker added a comment.
fixed requested changes.
i will commit it when i have access.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62399/new/
https://reviews.ll
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Looks like a good first step, thanks!
Comment at: clang/include/clang/AST/ASTContext.h:2821-2823
+ /// Adds an APValue that will be destructed during the destruction of the
Tyker added a comment.
Herald added a subscriber: rnkovacs.
ping @rsmith
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62399/new/
https://reviews.llvm.org/D62399
___
cfe-commits mailing list
cfe-commits@lists.llvm.or
Tyker created this revision.
Tyker added a reviewer: rsmith.
Herald added subscribers: llvm-commits, cfe-commits, dexonsmith, hiraditya.
Herald added a reviewer: martong.
Herald added a reviewer: shafik.
Herald added projects: clang, LLVM.
When using ConstantExpr we often need the result of the ex
