]> granicus.if.org Git - clang/commit
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)
commiteb0c09f9e0327ae48bb41c724faccfcf542b1b94
tree1bf852d1eb129b666c973f666f7096575da8f844
parenta989b78eff1cfe6911c5d1cca6a8e35f47386145
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.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@226383 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/CGCall.cpp
lib/CodeGen/CGDebugInfo.h