Thanks for doing this! It has been long time I work on big change for LLVM so sorry for forgetting that.
On Saturday, September 16, 2017, NAKAMURA Takumi via Phabricator < revi...@reviews.llvm.org> wrote: > chapuni added a comment. > > Fixed in https://reviews.llvm.org/rL313456. > > > > ================ > Comment at: include/llvm/IR/DiagnosticHandler.h:12 > +//===------------------------------------------------------ > ----------------===// > + > +#include "llvm/ADT/StringRef.h" > ---------------- > Did you forget include guard here? > > > https://reviews.llvm.org/D33514 > > > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
