[PATCH] D68869: [DebugInfo] Fix truncation of call site immediates

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 11 15:20:02 PDT 2019


aprantl added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp:623
       if (ParamValue->first.isImm()) {
-        unsigned Val = ParamValue->first.getImm();
+        auto Val = ParamValue->first.getImm();
         DbgValueLoc DbgLocVal(ParamValue->second, Val);
----------------
We should only use auto where the type is obvious from the context. Let's use uint64_t here.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D68869





More information about the llvm-commits mailing list