Skip to content
  1. Jun 02, 2008
  2. Jun 01, 2008
    • Owen Anderson's avatar
      Don't remove the memcpy when call slot substitution fails. · d071a870
      Owen Anderson authored
      llvm-svn: 51848
      d071a870
    • Duncan Sands's avatar
      When simplifying a call to a bitcast function, tighten up · 0397cd2e
      Duncan Sands authored
      the conditions for performing the transform when only the
      function declaration is available: no longer allow turning
      i32 into i64 for example.  Only allow changing between
      pointer types, and between pointer types and integers of
      the same size.  For return values ptr -> intptr was already
      allowed; I added ptr -> ptr and intptr -> ptr while there.
      As shown by a recent objc testcase, changing the way
      parameters/return values are passed can be fatal when calling
      code written in assembler that directly manipulates call
      arguments and return values unless the transform has no
      impact on the way they are passed at the codegen level.
      While it is possible to imagine an ABI that treats integers
      of pointer size differently to pointers, I don't think LLVM
      supports any so the transform should now be safe while still
      being useful.
      
      llvm-svn: 51834
      0397cd2e
  3. May 31, 2008
  4. May 30, 2008
  5. May 29, 2008
  6. May 28, 2008
  7. May 27, 2008
  8. May 26, 2008
  9. May 25, 2008
  10. May 24, 2008
  11. May 23, 2008
    • Dan Gohman's avatar
      Tidy up BasicBlock::getFirstNonPHI, and change a bunch of places to · f96e1371
      Dan Gohman authored
      use it instead of duplicating its functionality.
      
      llvm-svn: 51499
      f96e1371
    • Matthijs Kooijman's avatar
      f52b23c0
    • Matthijs Kooijman's avatar
      Restucture a part of the SimplifyCFG pass and include a testcase. · aef2b819
      Matthijs Kooijman authored
      The SimplifyCFG pass looks at basic blocks that contain only phi nodes,
      followed by an unconditional branch. In a lot of cases, such a block (BB) can
      be merged into their successor (Succ).
      
      This merging is performed by TryToSimplifyUncondBranchFromEmptyBlock. It does
      this by taking all phi nodes in the succesor block Succ and expanding them to
      include the predecessors of BB. Furthermore, any phi nodes in BB are moved to
      Succ and expanded to include the predecessors of Succ as well.
      
      Before attempting this merge, CanPropagatePredecessorsForPHIs checks to see if
      all phi nodes can be properly merged. All functional changes are made to
      this function, only comments were updated in
      TryToSimplifyUncondBranchFromEmptyBlock.
      
      In the original code, CanPropagatePredecessorsForPHIs looks quite convoluted
      and more like stack of checks added to handle different kinds of situations
      than a comprehensive check. In particular the first check in the function did
      some value checking for the case that BB and Succ have a common predecessor,
      while the last check in the function simply rejected all cases where BB and
      Succ have a common predecessor. The first check was still useful in the case
      that BB did not contain any phi nodes at all, though, so it was not completely
      useless.
      
      Now, CanPropagatePredecessorsForPHIs is restructured to to look a lot more
      similar to the code that actually performs the merge. Both functions now look
      at the same phi nodes in about the same order.  Any conflicts (phi nodes with
      different values for the same source) that could arise from merging or moving
      phi nodes are detected. If no conflicts are found, the merge can happen.
      
      Apart from only restructuring the checks, two main changes in functionality
      happened.
      
      Firstly, the old code rejected blocks with common predecessors in most cases.
      The new code performs some extra checks so common predecessors can be handled
      in a lot of cases. Wherever common predecessors still pose problems, the
      blocks are left untouched.
      
      Secondly, the old code rejected the merge when values (phi nodes) from BB were
      used in any other place than Succ. However, it does not seem that there is any
      situation that would require this check. Even more, this can be proven.
      
      Consider that BB is a block containing of a single phi node "%a" and a branch
      to Succ. Now, since the definition of %a will dominate all of its uses, BB
      will dominate all blocks that use %a. Furthermore, since the branch from BB to
      Succ is unconditional, Succ will also dominate all uses of %a.
      
      Now, assume that one predecessor of Succ is not dominated by BB (and thus not
      dominated by Succ). Since at least one use of %a (but in reality all of them)
      is reachable from Succ, you could end up at a use of %a without passing
      through it's definition in BB (by coming from X through Succ). This is a
      contradiction, meaning that our original assumption is wrong. Thus, all
      predecessors of Succ must also be dominated by BB (and thus also by Succ).
      
      This means that moving the phi node %a from BB to Succ does not pose any
      problems when the two blocks are merged, and any use checks are not needed.
      
      llvm-svn: 51478
      aef2b819
    • Matthijs Kooijman's avatar
      Indent fix. · f399bbf9
      Matthijs Kooijman authored
      llvm-svn: 51477
      f399bbf9
    • Nick Lewycky's avatar
      Constant integer vectors may also be negated. · 3bf5512d
      Nick Lewycky authored
      llvm-svn: 51476
      3bf5512d
    • Nick Lewycky's avatar
      Typo. · 8f3127c5
      Nick Lewycky authored
      llvm-svn: 51475
      8f3127c5
    • Nick Lewycky's avatar
      Revert X + X --> X * 2 optz'n which pessimizes heavily on x86. · 4f3d8785
      Nick Lewycky authored
      llvm-svn: 51474
      4f3d8785
    • Nick Lewycky's avatar
      Implement X + X for vectors. · 452fb329
      Nick Lewycky authored
      llvm-svn: 51472
      452fb329
    • Nick Lewycky's avatar
      Fix a recently added optimization to not crash on vectors. · 2ec9a011
      Nick Lewycky authored
      llvm-svn: 51471
      2ec9a011
    • Dan Gohman's avatar
      Generalize the new code in instcombine's ComputeNumSignBits for handling · 6d5f120c
      Dan Gohman authored
      and/or to handle more cases (such as this add-sitofp.ll testcase), and
      port it to selectiondag's ComputeNumSignBits.
      
      llvm-svn: 51469
      6d5f120c
    • Dan Gohman's avatar
      Use isSingleValueType instead of isFirstClassType to · 53b26985
      Dan Gohman authored
      exclude struct and array types.
      
      llvm-svn: 51467
      53b26985
    • Dale Johannesen's avatar
      Allow for switch with no cases. Was causing fault · fecb8824
      Dale Johannesen authored
      in gcc.dg/pr27531-1.c.
      
      llvm-svn: 51464
      fecb8824
    • Dan Gohman's avatar
      Use isSingleValueType instead of isFirstClassType to · 30ab45d0
      Dan Gohman authored
      exclude struct and array types.
      
      llvm-svn: 51459
      30ab45d0
    • Dan Gohman's avatar
      Use isSingleValueType instead of isFirstClassType to · 7a0566b9
      Dan Gohman authored
      exclude struct and array types.
      
      llvm-svn: 51456
      7a0566b9
  12. May 22, 2008
Loading