[SystemZ] Add long branch pass
authorRichard Sandiford <rsandifo@linux.vnet.ibm.com>
Mon, 20 May 2013 14:23:08 +0000 (14:23 +0000)
committerRichard Sandiford <rsandifo@linux.vnet.ibm.com>
Mon, 20 May 2013 14:23:08 +0000 (14:23 +0000)
commit44b486ed78c60b50aa14d4eed92ee828d4d44293
treeb1de5619d7dc7c9449ef44427e62650a4cccba29
parente932e891e55d90398f07957b7333b4ce66ae3829
[SystemZ] Add long branch pass

Before this change, the SystemZ backend would use BRCL for all branches
and only consider shortening them to BRC when generating an object file.
E.g. a branch on equal would use the JGE alias of BRCL in assembly output,
but might be shortened to the JE alias of BRC in ELF output.  This was
a useful first step, but it had two problems:

(1) The z assembler isn't traditionally supposed to perform branch shortening
    or branch relaxation.  We followed this rule by not relaxing branches
    in assembler input, but that meant that generating assembly code and
    then assembling it would not produce the same result as going directly
    to object code; the former would give long branches everywhere, whereas
    the latter would use short branches where possible.

(2) Other useful branches, like COMPARE AND BRANCH, do not have long forms.
    We would need to do something else before supporting them.

    (Although COMPARE AND BRANCH does not change the condition codes,
    the plan is to model COMPARE AND BRANCH as a CC-clobbering instruction
    during codegen, so that we can safely lower it to a separate compare
    and long branch where necessary.  This is not a valid transformation
    for the assembler proper to make.)

This patch therefore moves branch relaxation to a pre-emit pass.
For now, calls are still shortened from BRASL to BRAS by the assembler,
although this too is not really the traditional behaviour.

The first test takes about 1.5s to run, and there are likely to be
more tests in this vein once further branch types are added.  The feeling
on IRC was that 1.5s is a bit much for a single test, so I've restricted
it to SystemZ hosts for now.

The patch exposes (and fixes) some typos in the main CodeGen/SystemZ tests.
A later patch will remove the {{g}}s from that directory.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@182274 91177308-0d34-0410-b5e6-96231b3b80d8
17 files changed:
lib/Target/SystemZ/CMakeLists.txt
lib/Target/SystemZ/MCTargetDesc/SystemZMCAsmBackend.cpp
lib/Target/SystemZ/README.txt
lib/Target/SystemZ/SystemZ.h
lib/Target/SystemZ/SystemZISelLowering.cpp
lib/Target/SystemZ/SystemZInstrInfo.cpp
lib/Target/SystemZ/SystemZInstrInfo.h
lib/Target/SystemZ/SystemZInstrInfo.td
lib/Target/SystemZ/SystemZLongBranch.cpp [new file with mode: 0644]
lib/Target/SystemZ/SystemZMCInstLower.cpp
lib/Target/SystemZ/SystemZTargetMachine.cpp
test/CodeGen/SystemZ/Large/branch-range-01.py [new file with mode: 0644]
test/CodeGen/SystemZ/Large/branch-range-02.py [new file with mode: 0644]
test/CodeGen/SystemZ/Large/lit.local.cfg [new file with mode: 0644]
test/CodeGen/SystemZ/branch-05.ll
test/CodeGen/SystemZ/int-cmp-30.ll
test/CodeGen/SystemZ/int-cmp-31.ll