[PATCH] D43093: [FastISel] Sink local value materializations to first use
Matthias Braun via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 27 13:37:09 PST 2018
MatzeB added a comment.
> Now, should we go to such length to make the debug information be friendlier to some debugger, I am not sure, but I let you decide.
Definitely not! That debugger constraint feels like it is designed for ancient compilers producing code and register allocating for one C statement at a time.
Luckily this change makes sense independently of the debugger used IMO.
https://reviews.llvm.org/D43093
More information about the llvm-commits
mailing list