[PATCH] [opt] Hoist the call throuh SymbolBody::getReplacement out of the inline method to get a SymbolBody and into the callers, and kill now dead includes.

Chandler Carruth chandlerc at gmail.com
Fri Jun 26 20:40:38 PDT 2015


Hi ruiu,

This removes the need to have the SymbolBody definition when we're
defining the inline method and makes it a better inline method. That was
the only reason for a lot of header includes here. Removing these and
using forward declarations actually uncovers a bunch of cross-header
dependencies that I've fixed while I'm here, and will allow me to
introduce some *important* inline code into Chunks.h that requires the
definition of ObjectFile.

No functionality changed at this point.

http://reviews.llvm.org/D10789

Files:
  COFF/Chunks.cpp
  COFF/InputFiles.h
  COFF/SymbolTable.cpp
  COFF/SymbolTable.h

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10789.28623.patch
Type: text/x-patch
Size: 3604 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150627/2dc5595e/attachment.bin>


More information about the llvm-commits mailing list