[PATCH] D27471: [BDCE/DebugInfo] Preserve llvm.dbg.value's argument

[Davide Italiano via Phabricator via llvm-commits]

davide added inline comments.


================
Comment at: lib/Transforms/Scalar/BDCE.cpp:45
+    if (I.mayHaveSideEffects() && !I.hasNUsesOrMore(1))
+      continue;
+
----------------
aprantl wrote:
> It looks like this alters the behavior of the optimization beyond debug info. It would be great if someone else could sign off on this change.
I discussed this with David M. and he was OK with this solution. I'm still a little nervous this may break optimizations, so I'll be more confident to get this in if Hal signs it off.


https://reviews.llvm.org/D27471