RE: [clang] f978ea4 - [clang][clang-scan-deps] Aggregate the full dependency information.

[Nemanja Ivanovic via cfe-commits]

Hi Michael,

this turns out to be an assert failure:

 

clang/tools/clang-scan-deps/ClangScanDeps.cpp:340: const clang::tooling::dependencies::ModuleDeps& FullDeps::lookup
ModuleDeps(clang::tooling::dependencies::ClangModuleDep): Assertion `I != Modules.end()' failed.

 

Does this help with debugging at all?

 

Nemanja Ivanovic
LLVM PPC Backend Development
IBM Toronto Lab
Email: neman...@ca.ibm.com
Phone: 905-413-3388

 

 

----- Original message -----
From: Michael Spencer <bigchees...@gmail.com>
To: Nemanja Ivanovic <neman...@ca.ibm.com>
Cc: cfe-commits@lists.llvm.org, LLVM on Power <powerl...@ca.ibm.com>
Subject: [EXTERNAL] Re: [clang] f978ea4 - [clang][clang-scan-deps] Aggregate the full dependency information.
Date: Fri, Dec 13, 2019 4:44 PM
 

On Thu, Dec 12, 2019 at 4:23 AM Nemanja Ivanovic <neman...@ca.ibm.com> wrote:

Hi Michael,

We are happy to help troubleshoot the issue this caused on our bot. Unfortunately, this bot is not one where we can give you access so we'll have to try and work together to debug this.

Can you provide the link to the failing build so we can see which test case it was that caused the problem and we can start debugging from there?

 

Here's one of the failures: http://lab.llvm.org:8011/builders/clang-ppc64le-linux-lnt/builds/21884

 

It looks like it failed on every ppc64 linux bot (both be and le), but no other bots.

 

- Michael Spencer

 

 

Nemanja Ivanovic
LLVM PPC Backend Development
IBM Toronto Lab
Email: neman...@ca.ibm.com
Phone: 905-413-3388

 

 

----- Original message -----
From: Michael Spencer <bigchees...@gmail.com>
To: powerl...@ca.ibm.com
Cc: cfe-commits@lists.llvm.org
Subject: [EXTERNAL] Re: [clang] f978ea4 - [clang][clang-scan-deps] Aggregate the full dependency information.
Date: Wed, Dec 11, 2019 7:34 PM
 

On Wed, Dec 11, 2019 at 2:41 PM Michael Spencer via cfe-commits <cfe-commits@lists.llvm.org> wrote:

Author: Michael Spencer
Date: 2019-12-11T14:40:51-08:00
New Revision: f978ea498309adaebab8fbf1cd6e520e7e0e11f1

URL: https://github.com/llvm/llvm-project/commit/f978ea498309adaebab8fbf1cd6e520e7e0e11f1
DIFF: https://github.com/llvm/llvm-project/commit/f978ea498309adaebab8fbf1cd6e520e7e0e11f1.diff

LOG: [clang][clang-scan-deps] Aggregate the full dependency information.

Differential Revision: https://reviews.llvm.org/D70268

Added:


Modified:
    clang/include/clang/Tooling/DependencyScanning/DependencyScanningTool.h
    clang/include/clang/Tooling/DependencyScanning/ModuleDepCollector.h
    clang/lib/Tooling/DependencyScanning/DependencyScanningTool.cpp
    clang/lib/Tooling/DependencyScanning/DependencyScanningWorker.cpp
    clang/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp
    clang/test/ClangScanDeps/Inputs/modules_cdb.json
    clang/test/ClangScanDeps/modules-full.cpp
    clang/tools/clang-scan-deps/ClangScanDeps.cpp

 

 

Looks like this broke clang-ppc64be-linux. Is there a good way to debug this? It's not failing anywhere else, and none of the code should care about endianness.

 

I'll revert for now.

 

- Michael Spencer

 

 

 

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits