[PATCH] Move X86DisassemblerDecoderCommon.h to public includes

Reid Kleckner rnk at google.com
Thu Dec 26 10:14:14 PST 2013


  I don't think moving to include/llvm/TableGen is the right way to go.  It's not like the X86 disassembler links against a tablegen library.  The only correct layering without adding new libraries is moving it to include/llvm/Support, since both llvm-tblgen and the x86 target link against Support.  I don't really like that arrangement, but we already have these kind of enums-and-types-only headers like ELF.h and COFF.h in Support.

http://llvm-reviews.chandlerc.com/D2471



More information about the llvm-commits mailing list