[PATCH] D104394: [MachineCopyPropagation] Fix differences in code gen when compiling with -g
Jeremy Morse via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 21 05:01:49 PDT 2021
jmorse added subscribers: probinson, aprantl, dblaikie.
jmorse added a comment.
@predator5047 This patch is worth pursuing -- D98401 <https://reviews.llvm.org/D98401> involves codegen alterations and will necessarily take a long while to review; wheras this patch could land before llvm-13.
This patch is obviously effective at preventing codegen from being changed by debug-info; the downside is that the variable location is wrong afterwards (`DBG_VALUE $rcx` in the CHECK lines). I'm fairly confident we would prefer to have a false variable location over unstable code generation so it could land as-is; @aprantl @probinson @dblaikie would you agree?
To get the variable location right, you need to populate the `CopyDbgUsers` container, and re-write them when propagateDefs deletes the copy.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104394/new/
https://reviews.llvm.org/D104394
More information about the llvm-commits
mailing list