[PATCH] D48676: [Local] replaceAllDbgUsesWith: Update debug values before RAUW

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 5 13:17:30 PDT 2018


aprantl added inline comments.


================
Comment at: lib/Transforms/Utils/Local.cpp:1853
+
+      // Without knowing signedness, sign/zero extension isn't possible.
+      auto Signedness = Var->getSignedness();
----------------
There is no way to sign-extend, that's right, but zero-extension as a concept doesn't make much sense in the untyped DWARF (4) stack, since we don't know how large the stack values are anyway. Looks like the code knows this but the comment hasn't been updated yet.


https://reviews.llvm.org/D48676





More information about the llvm-commits mailing list