[PATCH] D43093: [FastISel] Sink local value materializations to first use

[Matthias Braun via Phabricator via llvm-commits]

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