[PATCH] D16263: [CodeGenPrepare] Also consider metadata uses

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 17 08:44:55 PST 2016


hfinkel added a comment.

In http://reviews.llvm.org/D16263#328811, @loladiro wrote:

> As far as I can tell, DWARF values are implicitly truncated. The problem is not the final DWARF representation but our IR representation. Passes like SROA rely on the sizing information of IR and debug information to match


Why does it make this assumption? Do other passes that widen variables (e.g. indvarsimplify) also trigger this problem?

> , which is what the upcoming Verifier stuff is supposed to ensure. Of course, SROA will never run after CodeGenPrepare, so it's not an actual problem, but it would be very annoying to not be able to have the Verifier check this invariant, because of this one thing. I do think it might be reasonable to have an (`OP_trunc size`) for the DIExpression. It could get ignored by the backend, but would allow us to do proper verification.





Repository:
  rL LLVM

http://reviews.llvm.org/D16263





More information about the llvm-commits mailing list