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

[Sanjoy Das via llvm-commits]

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