llvm-svn: 78679

llvm-svn: 78678

llvm-svn: 78677

other is a subclass of it is effectively handled by the prior tests.

llvm-svn: 78676

llvm-svn: 78675

llvm-svn: 78673

into one bigger test (which runs faster)

llvm-svn: 78672

llvm-svn: 78671

llvm-svn: 78670

MachineFunctionPass passes.

llvm-svn: 78669

alignment. Only the minimum alignment guaranteed by the ABI may be assumed.

llvm-svn: 78668

llvm-svn: 78667

llvm-svn: 78666

llvm-svn: 78665

llvm-svn: 78664

llvm-svn: 78663

llvm-svn: 78662

llvm-svn: 78661

llvm-svn: 78660

llvm-svn: 78659

match base only address, i.e. [r] since Thumb2 requires a offset register field.
For those, use [r + imm12] where the immediate is zero.
Note the generated assembly code does not look any different after the patch.
But the bug would have broken the JIT (if there is Thumb2 support) and it can
break later passes which expect the address mode to be well-formed.

llvm-svn: 78658

llvm-svn: 78657

llvm-svn: 78656

llvm-svn: 78655

llvm-svn: 78654

llvm-svn: 78652

llvm-svn: 78651

The register scavenger maintains a DistanceMap that maps MI pointers to their
distance from the top of the current MBB. The DistanceMap is built
incrementally in forward() and in bulk in findFirstUse(). It is used by
scavengeRegister() to determine which candidate register has the longest
unused interval.

Unfortunately the DistanceMap contents can become outdated. The first time
scavengeRegister() is called, the DistanceMap is filled to cover the MBB. If
then instructions are inserted in the MBB (as they always are following
scavengeRegister()), the recorded distances are too short. This causes bad
behaviour in the included test case where a register use /after/ the current
position is ignored because findFirstUse() thinks is is /before/ the current
position. A "using an undefined register" assertion follows promptly.

The fix is to build a fresh DistanceMap at the top of scavengeRegister(), and
discard it after use. This means that DistanceMap is no longer needed as a
RegScavenger member variable, and forward() doesn't need to update it.

The fix then discloses issue number two in the same test case: The candidate
search in scavengeRegister() finds a CSR that has been saved in the prologue,
but is currently unused. It would be both inefficient and wrong to spill such
a register in the emergency spill slot. In the present case, the emergency
slot restore is placed immediately before the normal epilogue restore, leading
to a "Redefining a live register" assertion.

Fix number two: When scavengerRegister() stumbles upon an unused register that
is overwritten later in the MBB, return that register early. It is important
to verify that the register is defined later in the MBB, otherwise it might be
an unspilled CSR.

llvm-svn: 78650

 - Since the function is never called in NDEBUG mode, just dropped the DEBUG()
   uses here.

llvm-svn: 78649

llvm-svn: 78648

the overloaded vector types allowed floating-point or integer vector elements.
Most of these operations actually depend on the element type, so bitcasting
was not an option.

If you include the vpadd intrinsics that I updated earlier, this gets rid
of 20 intrinsics.

llvm-svn: 78646

llvm-svn: 78643

llvm-svn: 78642

llvm-svn: 78641

llvm-svn: 78640

yet (I'm not even sure what they do).

llvm-svn: 78639

(and outputting a diagnostic pointing at the wrong place), all of which lead to
much confusion.

llvm-svn: 78637

 - This drops us to 123 ambiguous instructions (previously ~500) on X86.

llvm-svn: 78636

Passing of indirect arguments starts after return value on the callee's frame.

llvm-svn: 78635

llvm-svn: 78634