r230446 - MS ABI: Try to respect external AST source record layouts

Reid Kleckner rnk at google.com
Wed Feb 25 09:43:50 PST 2015


On Wed, Feb 25, 2015 at 3:35 AM, NAKAMURA Takumi <geek4civic at gmail.com>
wrote:

> I have reverted it (and r230448), for now. Reproducible with
> LLVM_DEFAULT_TARGET_TRIPLE=x86_64-win32.
>

I don't like to reconfigure and rebuild most of LLVM to repro these issues.
Is it possible to pass this to llvm-lit.py somehow to get %clang_cc1 to
expand to clang -cc1 -triple $mytriple?


> Unfortunately, clang/test/PCH/headersearch.cpp is skipped on Windows
> due to shell issue. Could you take a look please?
>

I can reproduce with LIT_USE_INTERNAL_SHELL=0. The problem is that this
test relies on -Wpadded, which is only implemented in the Itanium record
layout code. I think we can safely give headersearch.cpp a triple and
reland.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150225/e7591f57/attachment.html>


More information about the cfe-commits mailing list