bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.

LG. It's a bit sad but I guess the cases where that legitimately can happen due 
to template weirdness are far outweighed by the cases where we have a non-self 
contained header. The cases I've been thinking of are stuff like:

header.h:

  template <typename T>
  class Foo {
    T t;
  };

test.cc:

  #include "header.h"
  
  class Bar;
  Foo<Bar> f;

Here we'd have to include the header for Bar into test.cc, but figuring that 
out is hard. Ignoring all of that for now is fine.


http://reviews.llvm.org/D20950



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to