[clang] [clang-tools-extra] [lldb] [clang][NFC] Remove `macro_begin` and `macro_end` from `Preprocessor` (PR #197817)

[David Stone via cfe-commits]

================
@@ -55,7 +56,7 @@ void BadSignalToKillThreadCheck::check(const MatchFinder::MatchResult &Result) {
     return IntValue.getZExtValue();
   };
 
-  const auto SigtermMacro = llvm::find_if(PP->macros(), IsSigterm);
+  const auto SigtermMacro = llvm::find_if(Macros, IsSigterm);
----------------
davidstone wrote:

Yeah, I noticed that, too. I was considering a follow-up PR for that, but I have a bunch of other stuff that I'll be getting to first.

https://github.com/llvm/llvm-project/pull/197817