Skip to content
  1. Nov 07, 2012
  2. Nov 06, 2012
  3. Nov 03, 2012
  4. Nov 02, 2012
  5. Nov 01, 2012
    • Chandler Carruth's avatar
      Revert the majority of the next patch in the address space series: · 5da3f051
      Chandler Carruth authored
      r165941: Resubmit the changes to llvm core to update the functions to
               support different pointer sizes on a per address space basis.
      
      Despite this commit log, this change primarily changed stuff outside of
      VMCore, and those changes do not carry any tests for correctness (or
      even plausibility), and we have consistently found questionable or flat
      out incorrect cases in these changes. Most of them are probably correct,
      but we need to devise a system that makes it more clear when we have
      handled the address space concerns correctly, and ideally each pass that
      gets updated would receive an accompanying test case that exercises that
      pass specificaly w.r.t. alternate address spaces.
      
      However, from this commit, I have retained the new C API entry points.
      Those were an orthogonal change that probably should have been split
      apart, but they seem entirely good.
      
      In several places the changes were very obvious cleanups with no actual
      multiple address space code added; these I have not reverted when
      I spotted them.
      
      In a few other places there were merge conflicts due to a cleaner
      solution being implemented later, often not using address spaces at all.
      In those cases, I've preserved the new code which isn't address space
      dependent.
      
      This is part of my ongoing effort to clean out the partial address space
      code which carries high risk and low test coverage, and not likely to be
      finished before the 3.2 release looms closer. Duncan and I would both
      like to see the above issues addressed before we return to these
      changes.
      
      llvm-svn: 167222
      5da3f051
    • Chandler Carruth's avatar
      Revert the series of commits starting with r166578 which introduced the · 7ec5085e
      Chandler Carruth authored
      getIntPtrType support for multiple address spaces via a pointer type,
      and also introduced a crasher bug in the constant folder reported in
      PR14233.
      
      These commits also contained several problems that should really be
      addressed before they are re-committed. I have avoided reverting various
      cleanups to the DataLayout APIs that are reasonable to have moving
      forward in order to reduce the amount of churn, and minimize the number
      of commits that were reverted. I've also manually updated merge
      conflicts and manually arranged for the getIntPtrType function to stay
      in DataLayout and to be defined in a plausible way after this revert.
      
      Thanks to Duncan for working through this exact strategy with me, and
      Nick Lewycky for tracking down the really annoying crasher this
      triggered. (Test case to follow in its own commit.)
      
      After discussing with Duncan extensively, and based on a note from
      Micah, I'm going to continue to back out some more of the more
      problematic patches in this series in order to ensure we go into the
      LLVM 3.2 branch with a reasonable story here. I'll send a note to
      llvmdev explaining what's going on and why.
      
      Summary of reverted revisions:
      
      r166634: Fix a compiler warning with an unused variable.
      r166607: Add some cleanup to the DataLayout changes requested by
               Chandler.
      r166596: Revert "Back out r166591, not sure why this made it through
               since I cancelled the command. Bleh, sorry about this!
      r166591: Delete a directory that wasn't supposed to be checked in yet.
      r166578: Add in support for getIntPtrType to get the pointer type based
               on the address space.
      llvm-svn: 167221
      7ec5085e
    • Owen Anderson's avatar
      b351c8d6
    • Jakob Stoklund Olesen's avatar
      Exploit the new identity composition in composeSubRegIndices(). · 9892a4b7
      Jakob Stoklund Olesen authored
      The static compose() function in RegisterCoalescer was doing the exact
      same thing.
      
      llvm-svn: 167198
      9892a4b7
  6. Oct 31, 2012
  7. Oct 30, 2012
  8. Oct 29, 2012
    • Ulrich Weigand's avatar
      In various places throughout the code generator, there were special · 3abb3438
      Ulrich Weigand authored
      checks to avoid performing compile-time arithmetic on PPCDoubleDouble.
      
      Now that APFloat supports arithmetic on PPCDoubleDouble, those checks
      are no longer needed, and we can treat the type like any other.
      
      llvm-svn: 166958
      3abb3438
    • Jakob Stoklund Olesen's avatar
      Completely disallow partial copies in adjustCopiesBackFrom(). · 9a06696a
      Jakob Stoklund Olesen authored
      Partial copies can show up even when CoalescerPair.isPartial() returns
      false. For example:
      
         %vreg24:dsub_0<def> = COPY %vreg31:dsub_0; QPR:%vreg24,%vreg31
      
      Such a partial-partial copy is not good enough for the transformation
      adjustCopiesBackFrom() needs to do.
      
      llvm-svn: 166944
      9a06696a
    • Duncan Sands's avatar
      Remove a wrapper around getIntPtrType added to GVN by Hal in commit 166624 (the · 5bdd9dda
      Duncan Sands authored
      wrapper returns a vector of integers when passed a vector of pointers) by having
      getIntPtrType itself return a vector of integers in this case.  Outside of this
      wrapper, I didn't find anywhere in the codebase that was relying on the old
      behaviour for vectors of pointers, so give this a whirl through the buildbots.
      
      llvm-svn: 166939
      5bdd9dda
    • Preston Gurd's avatar
      This patch addresses a problem with the Post RA scheduler generating an · 52dacca9
      Preston Gurd authored
      incorrect instruction sequence due to it not being aware that an
      inline assembly instruction may reference memory.
      
      This patch fixes the problem by causing the scheduler to always assume that any
      inline assembly code instruction could access memory. This is necessary because
      the internal representation of the inline instruction does not include
      any information about memory accesses.
       
      This should fix PR13504.
      
      llvm-svn: 166929
      52dacca9
    • Lang Hames's avatar
      Remove unused typedef. · ee6142c3
      Lang Hames authored
      llvm-svn: 166910
      ee6142c3
  9. Oct 27, 2012
  10. Oct 26, 2012
  11. Oct 25, 2012
  12. Oct 24, 2012
Loading