Re: r301707 - Adapt to LLVM API change (DINamespace no longer takes line/file info).

2017-05-01 Thread Adrian Prantl via cfe-commits
I relaxed the testcase to allow for the extra import in r301805. The extra import is for the anonymous namespace that I added to the testcase, which is the intended behavior on the PS4 platform (DebugExplicitImport). -- adrian > On Apr 28, 2017, at 9:37 PM, Yung, Douglas wrote: > > Hi Adrian,

RE: r301707 - Adapt to LLVM API change (DINamespace no longer takes line/file info).

2017-04-28 Thread Yung, Douglas via cfe-commits
Hi Adrian, This commit, or the previous one you made seems to be causing a failure in the clang test CodeGenCXX\debug-info-namespace.cpp on the ps4 Windows and Linux bots. It's failing when trying to match the CHECK line on line 79 of the test because the line it matches seems to have one extra