nickdesaulniers added inline comments.
================
Comment at: clang/lib/Sema/SemaExpr.cpp:4738-4739
+ bool ConversionOk = tryConvertToTy(*this, Context.getSizeType(),
&ConvExpr);
+ assert(ConversionOk &&
+ "should be able to convert any integer type to size type");
+ return ConvExpr.get();
----------------
In a release build:
```
llvm-project/clang/lib/Sema/SemaExpr.cpp:4737:10: warning: unused variable
'ConversionOk' [-Wunused-variable]
bool ConversionOk = tryConvertToTy(*this, Context.getSizeType(), &ConvExpr);
^
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D76791/new/
https://reviews.llvm.org/D76791
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
