[PATCH] D125721: [CodeView] Combine variable def ranges that are continuous.

[Reid Kleckner via Phabricator via llvm-commits]

rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm, thanks!



================
Comment at: llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.h:506
+  static inline CodeViewDebug::LocalVarDef getEmptyKey() {
+    return CodeViewDebug::LocalVarDef::createFromOpaqueValue(~0UL);
+  }
----------------
You may need to use `ULL` sufixes so that this compiles as you expect on LLP64 platforms, such as Windows x64.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125721