[llvm] 393daca - [ARM] Enable TypePromotion by default

Sam Parker via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 05:20:38 PST 2019


Hi Hans,

The reproducer in the other ticket was enough and now that large preprocessed file will compile without an assertion.

Thanks,
Sam


Sam Parker

Compilation Tools Engineer | Arm

. . . . . . . . . . . . . . . . . . . . . . . . . . .

Arm.com

________________________________
From: Hans Wennborg <hans at chromium.org>
Sent: 10 December 2019 11:38
To: Sam Parker <Sam.Parker at arm.com>; Sam Parker <llvmlistbot at llvm.org>
Cc: llvm-commits <llvm-commits at lists.llvm.org>
Subject: Re: [llvm] 393daca - [ARM] Enable TypePromotion by default

On Mon, Dec 9, 2019 at 9:46 AM Hans Wennborg <hans at chromium.org> wrote:
>
> This caused the Chromium build to fail with an  "Too many bits for
> uint64_t" assertion. There's a reproducer at
> https://crbug.com/1031978#c2 and I'll follow up with a creduced
> version soon.

It turns out creduce is taking forever on this one. Since you already
sent out a fix (https://reviews.llvm.org/D71200), I'm guessing the
creduced repro isn't necessary anymore?

Thanks,
Hans
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191210/86e7f4e7/attachment.html>


More information about the llvm-commits mailing list