<div dir="ltr">Hmm.  No, I was using a clean tree.  I tried a50e63b38b931d945f97eac882278068221eca17 and the commit immediately before, and the build problem showed up at a50e63b38b931d945f97eac882278068221eca17.<div><br></div><div>Let me re-apply the patch locally and try both gcc and clang (as the host compilers).</div><div><br></div><div>Kazu Hirata</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Oct 5, 2023 at 1:26 PM Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Oct 5, 2023 at 4:19 PM Aaron Ballman <<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>> wrote:<br>
><br>
> On Thu, Oct 5, 2023 at 4:18 PM Aaron Ballman <<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>> wrote:<br>
> ><br>
> > On Thu, Oct 5, 2023 at 4:08 PM Kazu Hirata via cfe-commits<br>
> > <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
> > ><br>
> > ><br>
> > > Author: Kazu Hirata<br>
> > > Date: 2023-10-05T13:08:24-07:00<br>
> > > New Revision: a6acf3fd49a20c570a390af2a3c84e10b9545b68<br>
> > ><br>
> > > URL: <a href="https://github.com/llvm/llvm-project/commit/a6acf3fd49a20c570a390af2a3c84e10b9545b68" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/a6acf3fd49a20c570a390af2a3c84e10b9545b68</a><br>
> > > DIFF: <a href="https://github.com/llvm/llvm-project/commit/a6acf3fd49a20c570a390af2a3c84e10b9545b68.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/a6acf3fd49a20c570a390af2a3c84e10b9545b68.diff</a><br>
> > ><br>
> > > LOG: Revert "Fixes and closes #53952. Setting the ASTHasCompilerErrors member variable correctly based on the PP diagnostics. (#68127)"<br>
> > ><br>
> > > This reverts commit a50e63b38b931d945f97eac882278068221eca17.<br>
> > ><br>
> > > With clang-14.0.6 as the host compiler, I'm getting:<br>
> > ><br>
> > > ld.lld: error: undefined symbol: clang::ASTWriter::WriteAST(clang::Sema&, llvm::StringRef, clang::Module*, llvm::StringRef, bool, bool)<br>
> > > >>> referenced by ASTUnit.cpp<br>
> > > >>>               ASTUnit.cpp.o:(clang::ASTUnit::serialize(llvm::raw_ostream&)) in archive lib/libclangFrontend.a<br>
> ><br>
> > That's expected; we removed a parameter that appeared to be unused and<br>
> > was causing problems. Is lld using the parameter that was removed? If<br>
> > so, what was the intent of overriding the diagnostics engine?<br>
><br>
> Oh. I see, it wasn't lld using it, it comes from a missed usage in<br>
> ASTUnit.cpp that should have been updated.<br>
<br>
Nope, I'm back to being baffled. That code was updated in the PR and<br>
no bots went red with the change. We may need more details in order to<br>
address this; alternatively, did you have local changes to ASTUnit.cpp<br>
so that the file didn't get updated for you?<br>
<br>
~Aaron<br>
</blockquote></div>