[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.


More information about the llvm-commits mailing list