nridge added inline comments.

================
Comment at: clang-tools-extra/clangd/Hover.cpp:396
 // -2    => 0xfffffffe
-// -2^32 => 0xfffffffeffffffff
+// -2^32 => 0xffffffff00000000
 static llvm::FormattedNumber printHex(const llvm::APSInt &V) {
----------------
Just to make sure I'm not missing anything: this comment change is just fixing 
a pre-existing mistake in the comment (-2^32 was already printed as 
0xffffffff00000000), right? i.e. this patch doesn't change how any number is 
printed


================
Comment at: clang-tools-extra/clangd/unittests/HoverTests.cpp:2953
+    constexpr unsigned long value = -1; // wrap around
+    void foo() { va$1^lue; }
+  )cpp");
----------------
you can just write `va^lue` and access it as `T.point()`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146874

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

Reply via email to