[llvm] r315044 - Roll forward r314928

[Xinliang David Li via llvm-commits]

I have a release build Clang (built also with ThinLTO) which also compiles
it fine.

David

On Fri, Oct 6, 2017 at 4:23 PM, Xinliang David Li <davidxl at google.com>
wrote:

>
>
> On Fri, Oct 6, 2017 at 4:08 PM, Reid Kleckner <rnk at google.com> wrote:
>
>> On Fri, Oct 6, 2017 at 2:50 PM, Xinliang David Li <davidxl at google.com>
>> wrote:
>>
>>> On Fri, Oct 6, 2017 at 2:20 PM, Reid Kleckner <rnk at google.com> wrote:
>>>
>>>> This appears to have caused a bootstrap failure on Windows:
>>>> http://lab.llvm.org:8011/builders/clang-x64-ninja-win7/builds/6884
>>>> http://lab.llvm.org:8011/builders/clang-x86-windows-msvc2015
>>>> /builds/7611
>>>>
>>>> I am going to revert and try to reproduce. It looks like clang ends up
>>>> emitting an error on valid code.
>>>>
>>>
>>> Please help with the reproducible -- it will be hard for me to get a
>>> repro on windows.
>>>
>>
>> Can you build clang with your patch and try to compile this code?
>>
>> char *f(char *p, int i) {
>>   return p + i;
>> }
>>
>> If clang says:
>> t.c:2:12: error: invalid operands to binary expression ('char *' and
>> 'int')
>>   return p + i;
>>          ~ ^ ~
>> 1 error generated.
>>
>> Then you have the miscompile locally and it doesn't require Windows.
>>
>
>
>
> How is the self build compiler built? I have one built with
> RelWithDebInfo, and that compiler compiles this case fine without an error.
>
> David
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171006/3b8560f6/attachment.html>