[PATCH] New SystemZ backend: LLVM code changes

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri Apr 26 05:04:58 PDT 2013


> Thanks for the pointer.  How does this look?  In case anyone's
> wondering, I didn't add an equivalent of DebugInfo/AArch64/cfi-frame.ll
> because .cfi_* directives are already tested in the CodeGen/SystemZ/frame-*.ll
> tests.

Why -filetype=obj test? Please split these in two:

* A .ll to .s test that checks that the expected .cfi_* directives are produced
* A .s to .o test that checks that .cfi_ directives are correctly assembled.

> The tests exposed a bug too. :-)  I'll send a fixed backend patch
> if the tests look OK.
>
> Richard
>

Cheers,
Rafael



More information about the llvm-commits mailing list