Skip to content
  1. Oct 31, 2013
    • Rui Ueyama's avatar
      Revert "[PECOFF] Add atoms to the PassManager file" · bad4565b
      Rui Ueyama authored
      This reverts commit r193479.
      
      The atoms are already added to the file, so re-adding them caused the YAML
      writer to write the same atoms twice. That made the YAML reader to fail with
      "duplicate atom name" error.
      
      This is not the only error we've got for RoundTripYAMLPass for PECOFF, so we
      cannot enable the test yet. More fixes will come.
      
      Differential Revision: http://llvm-reviews.chandlerc.com/D2069
      
      llvm-svn: 193762
      bad4565b
  2. Oct 30, 2013
  3. Oct 29, 2013
  4. Oct 28, 2013
  5. Oct 27, 2013
    • Chandler Carruth's avatar
      Rather than rendering an argument to a std::string (which is primarily · 9204c831
      Chandler Carruth authored
      intended for debugging and diagnostic output), just inspect the spelling
      to check for specific prefixes in drectve section flags.
      
      In addition to being significantly cheaper and not relying on
      a debugging interface, this also avoids creating a temporary string and
      binding it to StringRef variable. We then went on to access it after the
      memory had been deallocated.
      
      This bug too was caught by ASan. I love ASan so much. =]
      
      llvm-svn: 193487
      9204c831
    • Chandler Carruth's avatar
      Use equals_lower() rather than creating a temporary string with lower(). · 50e5b329
      Chandler Carruth authored
      More important than any performance concerns, the code was dropping the
      temporary string on the floor after assigning it to a StringRef, and
      then used the StringRef later. Caught by running the LLD tests under
      ASan.
      
      llvm-svn: 193486
      50e5b329
  6. Oct 26, 2013
  7. Oct 25, 2013
  8. Oct 24, 2013
Loading