shivanshu3 added a comment.

> Can you get at least some testcase, even if not small?

Yes, in a few days I can get you the test case. I'll have to figure out what 
our company policy is for sharing code and if I should obfuscate it so I don't 
get into trouble :)

> MSVC does some kind of template instantiation for PCHs too

MSVC's cl.exe compiles the precomp header with an empty .cpp file, which is 
where the template instantiations occur, and which is also the reason why the 
precomp header has to be self contained. So yes, in principle your change 
should be safe when enabled by default for clang-cl but there is a bug 
somewhere which we should figure out.

@llunak


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88680/new/

https://reviews.llvm.org/D88680

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

Reply via email to