[PATCH] D31440: PR32382: Adapt to LLVM changes in DIExpression.

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 29 10:50:16 PDT 2017


dblaikie added a comment.

I'm a bit confused - the alloca was only emitted at -O0, by the looks of it. Presumably it's pessimizing in some way at higher optimization levels? Or is that not the case?

Also, it looks like this change lost the "if > gmlt" test, so might cause variable declarations (& thus types) to leak into GMLT, which is undesirable.


https://reviews.llvm.org/D31440





More information about the cfe-commits mailing list