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

Anastasis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 4 05:55:36 PDT 2018


gramanas added inline comments.


================
Comment at: lib/Transforms/Utils/Local.cpp:1844
+    // access the low `FromBits` bits when inspecting the source variable.
+    if (FromBits < ToBits)
+      return rewriteDebugUsers(From, To, DomPoint, DT, Identity);
----------------
Could FromBits be equal to ToBits? In that case neither zext nor sext is needed. Could this be `if (FromBits <= ToBits)`?


https://reviews.llvm.org/D48676





More information about the llvm-commits mailing list