Endilll wrote: > > @dougsonos Maybe you’ve just not pushed that change yet, but remember to > > put an entry for `SemaFunctionEffects.cpp` in the table of contents at the > > start of the `Sema` class definition. > > Thanks. I started to do that and got distracted by a bunch of sections which > aren't listed in the TOC :-/ Should I fix that while I'm there? (e.g. API > Notes, Bounds Safety)
Ideally you shouldn't include unrelated changes in your patch. I fixed the table of contents in 9f1dc01e0ab4f91c4052a712ce590d2e86b81dc3, so you should be good to go after you merge latest `main`. https://github.com/llvm/llvm-project/pull/99656 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
