Sirraide 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. https://github.com/llvm/llvm-project/pull/99656