rsmith added inline comments.

================
Comment at: clang/lib/AST/ExprConstant.cpp:9617
+    APFloat FPVal(0.0);
+    APSInt IVal(Info.Ctx.getIntWidth(E->getType()), 0);
+    bool isExact = true;
----------------
Please use `/*isUnsigned=*/false` for the second argument rather than `0`.


================
Comment at: clang/lib/AST/ExprConstant.cpp:9618
+    APSInt IVal(Info.Ctx.getIntWidth(E->getType()), 0);
+    bool isExact = true;
+
----------------
Please capitalize local variable names to match the convention used in this 
file.


================
Comment at: clang/lib/AST/ExprConstant.cpp:9625
+
+    return Success(IVal.getExtValue(), E);
+  }
----------------
Just `return Success(IVal, E);` here. Going via `getExtValue` will assert if 
the integer type is wider than 64 bits.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66862/new/

https://reviews.llvm.org/D66862



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to