[PATCH] D18429: Add lowering support for llvm.experimental.deoptimize

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 31 19:59:26 PDT 2016


Hi,

I've re-enabled the assertion in rL265099, and filed PR27172 for the 
webjit_jscc issue.

-- Sanjoy

Sanjoy Das wrote:
> sanjoy added a comment.
>
> Justin Lebar wrote:
>   >  Hi, I think this may be broken.
>   >
>   >  In particular, I tried to land http://reviews.llvm.org/rL265092,
>   >  which turns on an assertion, but had to back it out because it asserts
>   >  on the test added here.  (The assertion is currently only run with
>   >  -debug, which is doubly bad, because now your code can break for
>   >  unrelated reasons while you're trying to debug it!)
>
> Looks like webkit_jscc does not work with anything but an i64 return
> type.  The following crashes llc, for instance:
>
> target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
> target triple = "x86_64-apple-macosx10.11.0"
>
> declare webkit_jscc i32 @foo()
>
> define i32 @caller() {
> entry:
>
>    %v = call webkit_jscc i32 @foo()
>    ret i32 %v
>
> }
>
> CC'ing Andy and Juergen, since they might know.  For now I'll switch
> the test to return i64 and turn on the assertion.
>
>> Would you mind having a look?  Bonus points if you reland
>   >  http://reviews.llvm.org/rL265092 after fixing the issue.
>
> - Sanjoy
>
>
> Repository:
>    rL LLVM
>
> http://reviews.llvm.org/D18429
>
>
>


More information about the llvm-commits mailing list