[PATCH] D44761: Fix PR36793

Rafael Avila de Espindola via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 22 10:48:49 PDT 2018


espindola added inline comments.


================
Comment at: ELF/InputFiles.h:20
 #include "llvm/ADT/STLExtras.h"
+#include "llvm/DebugInfo/DWARF/DWARFDebugLine.h"
 #include "llvm/IR/Comdat.h"
----------------
jhenderson wrote:
> Do we need this header here? All I see that has changed is the use of pointers in a vector, which can be forward declared, can't they?
I don't think it is possible to forward declare an inner class (llvm::DWARFDebugLine::LineTable).



https://reviews.llvm.org/D44761





More information about the llvm-commits mailing list