Recommit r225083 (reverted in r225361) now that calls to aggregate initializers from...
authorDavid Blaikie <dblaikie@gmail.com>
Sun, 18 Jan 2015 00:14:21 +0000 (00:14 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Sun, 18 Jan 2015 00:14:21 +0000 (00:14 +0000)
commit7d2a2ac57b3485ec7330ee42dc7984751a7bdfde
tree9444d9f9d13b13346cbb879f0fdadc4a9a819241
parenta81d410d4f397a35054f4056ee4c032b6b43c1d1
Recommit r225083 (reverted in r225361) now that calls to aggregate initializers from in class non-static data members are explicitly attributed to the desired line.

The code setting the debug location being removed here was accidentally
leaking a location into the call to the non-static data member's ctor
call. Without it the call had no location and could cause assertion
failures if it was inlined. Now that it has a location (and a correct
one at that) this code should hopefully be no longer needed.

It's possible of course that other parts of the debug info are also
relying on the debug locations being set here to leak to where they're
needed - so we might see the same assertions again & will have to
investigate what the dependence was/is. But the chances are good that
any of those are debug info line table quality bugs we've just not found
yet anyway - so it'll be good to flush them out.

llvm-svn: 226383
clang/lib/CodeGen/CGCall.cpp
clang/lib/CodeGen/CGDebugInfo.h