erichkeane wrote: > > I see the 2nds commit doesn't add any tests! Please make it do so > > I've tried and I'm not quite sure how to do it. The issue is we need to test > AST printing of «implicitly declared» functions, such as builtins and > functions instrumented by lldb, such as `log` for logging; which seems > contradictory. Do you have any suggestions? > > Furthermore, the second commit adds an assertion of an invariant and then > fixes the places in the code where the invariant breaks. Is it fair to say > that the codebase itself is the test for the invariant? If any codepath would > break the invariant, the assertion would be triggered. Not on a release build > though, I imagine.
Hmm... I don't have a great idea. Is using it not enough to get it to be emitted? I see here we don't seem to cause them to be emitted: https://godbolt.org/z/nYzMca7Te Perhaps @AaronBallman has an idea. https://github.com/llvm/llvm-project/pull/93913 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
