[llvm-testresults] buildbot failure in lab.llvm.org on phase1 - sanity

llvmlab-buildmaster at lab.llvm.org llvmlab-buildmaster at lab.llvm.org
Wed Feb 26 16:28:34 PST 2014


The Buildbot has detected a new failure on builder phase1 - sanity while building cfe.
Full details are available at:
 http://lab.llvm.org:8013/builders/phase1%20-%20sanity/builds/17141

Buildbot URL: http://lab.llvm.org:8013/

Buildslave for this Build: macpro1

Build Reason: scheduler
Build Source Stamp: 202320
Blamelist: adrian,rnk

BUILD FAILED: failed

sincerely,
 -The Buildbot


================================================================================

CHANGES:
File: sret.cpp
On: http://10.1.1.2/svn/llvm-project
For: debuginfo-tests
At: Wed 26 Feb 2014 16:11:52
Changed By: adrian
Comments: Relax testcase: Newer versions of LLDB don't necessarily print the "{" either.Properties: 




Files:
 lib/CodeGen/CGRecordLayoutBuilder.cpp
 test/CodeGenCXX/pr18962.cpp
On: http://10.1.1.2/svn/llvm-project
For: cfe
At: Wed 26 Feb 2014 16:11:52
Changed By: rnk
Comments: PR18962: Fix "Unable to find record layout information for type"

Clang is using llvm::StructType::isOpaque() as a way of signaling if
we've finished record type conversion in
CodeGenTypes::isRecordLayoutComplete().  However, Clang was setting the
body of the type before it finished laying out the type as a base type.
Laying out the %class.C.base LLVM type attempts to convert more types,
eventually recursively attempting to layout 'C' again, at which point we
would say that layout was complete, even though we were still in the
middle of it.

By not setting the body, we correctly signal that layout is not
complete, and things work as expected.

At some point, it might be worth refactoring this to avoid looking at
the LLVM IR types under construction.Properties: 




LOGS:






More information about the llvm-testresults mailing list