[PATCH] D25611: When moving a zext near to its associated load, do not retain the origial debug location.

Adrian Prantl via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 14 09:51:14 PDT 2016


aprantl added a comment.

This feels a bit like a hack; but I'm not sure I have a better solution.

Would it be possible to post source code for this function? If the frontend has assigned an explicit location to the zext instruction, I would probably want that to be preserved (unless the load and the zext are combined I guess). If the zext doesn't appear in the source code, wouldn't it be more appropriate for the frontend to assign an artificial (line 0) location to the instruction?


https://reviews.llvm.org/D25611





More information about the llvm-commits mailing list