This is an archive of the discontinued LLVM Phabricator instance.

Differential D49905

[RISCV] Fix test after new support of "interrupt" attribute landed
AbandonedPublic

Authored by rogfer01 on Jul 27 2018, 12:03 AM.

Details

Reviewers
apazos
mgrang
asb
Summary

This test fails now because the attribute needs a value. I don't think for this particular test a specific value is relevant so I just went with user.

Diff Detail

Event Timeline

rogfer01 created this revision.Jul 27 2018, 12:03 AM
Herald added subscribers: rkruppe, the_o, brucehoult and 10 others. · View Herald TranscriptJul 27 2018, 12:03 AM
rogfer01 abandoned this revision.EditedJul 30 2018, 11:05 PM

Fixed in rL338341

Revision Contents

PathSize
test/
CodeGen/
RISCV/
2 lines

Diff 157649

test/CodeGen/RISCV/tail-calls.ll

Show First 20 Lines • Show All 100 Lines • ▼ Show 20 Lines
define void @caller_irq() #0 { define void @caller_irq() #0 {
; CHECK-LABEL: caller_irq ; CHECK-LABEL: caller_irq
; CHECK-NOT: tail callee_irq ; CHECK-NOT: tail callee_irq
; CHECK: call callee_irq ; CHECK: call callee_irq
entry: entry:
tail call void @callee_irq() tail call void @callee_irq()
ret void ret void
} }
attributes #0 = { "interrupt" } attributes #0 = { "interrupt"="user" }
; Byval parameters hand the function a pointer directly into the stack area ; Byval parameters hand the function a pointer directly into the stack area
; we want to reuse during a tail call. Do not tail call optimize functions with ; we want to reuse during a tail call. Do not tail call optimize functions with
; byval parameters. ; byval parameters.
declare i32 @callee_byval(i32** byval %a) declare i32 @callee_byval(i32** byval %a)
define i32 @caller_byval() { define i32 @caller_byval() {
; CHECK-LABEL: caller_byval ; CHECK-LABEL: caller_byval
; CHECK-NOT: tail callee_byval ; CHECK-NOT: tail callee_byval
Show All 31 Lines