[PATCH] New SystemZ backend: LLVM code changes

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri Apr 26 07:11:36 PDT 2013


> We need more than just .cfi_* directives if we want to keep testing the
> DW_CFA_advance_loc bits.  I suppose I could change the test so that it
> has a zero-sized function with CFI info like:
>
> #   DW_CFA_offset: r13 at cfa-56
> #   DW_CFA_offset: r14 at cfa-48
> #   DW_CFA_offset: r15 at cfa-40
> #   DW_CFA_def_cfa_offset: 320
>
> and no DW_CFA_advance_locs, but in practice no function is going
> to look like that.  (The allocation and storing are done by
> separate instructions.)
>
> This version removes the calls and the .gcc_except_table contents though.

OK, that looks good. The tests we have for DW_CFA_advance_loc in tree
just use nops. I would probably do the same in here, but keeping
"real" instructions is ok if you want.

Cheers,
Rafael



More information about the llvm-commits mailing list