aaron.ballman accepted this revision.
aaron.ballman added a comment.
In D137240#3909034 <https://reviews.llvm.org/D137240#3909034>, @tbaeder wrote:
> It's not really all that interesting since the current interpreter already
> does all the work, but it works like this.
Ahhh okay, that makes sense, thanks!
================
Comment at: clang/lib/AST/Interp/ByteCodeExprGen.cpp:461
+ return this->emitConst(Size.getQuantity(), E);
+ } else if (Kind == UETT_AlignOf || Kind == UETT_PreferredAlignOf) {
+ CharUnits Size;
----------------
No need for the `else` because of the unconditional return in the `if` branch.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137240/new/
https://reviews.llvm.org/D137240
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
