akyrtzi added a comment.

I'd like if we only had to use one flag (`-fallow-pcm-with-compiler-errors`) 
and have it handle both modules and PCH. Could you make the flag also work for 
PCH and/or add a test that verifies it works?
You may only have to change

  Opts.AllowPCHWithCompilerErrors = Args.hasArg(OPT_fallow_pch_with_errors);

to

  Opts.AllowPCHWithCompilerErrors = Args.hasArg(OPT_fallow_pch_with_errors, 
OPT_fallow_pcm_with_errors);


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

https://reviews.llvm.org/D91580

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

Reply via email to