[PATCH] D93222: [RFC][analyzer] Introduce MacroExpansionContext to libAnalysis

[Balázs Benics via Phabricator via cfe-commits]

steakhal updated this revision to Diff 314857.
steakhal marked 11 inline comments as done.
steakhal added a comment.

- move `MacroExpansionRangeRecorder` to `clang::detail` and mark it as a friend class
- fix comment typo in `getExpandedMacroForLocation`
- rename `getExpandedMacroForLocation` -> `getExpandedText`
- rename `getSubstitutedTextForLocation` -> `getOriginalText`
- introduce `llvm_unreachable` where applicable.

No tests were changed.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93222/new/

https://reviews.llvm.org/D93222

Files:
  clang/include/clang/Analysis/MacroExpansionContext.h
  clang/lib/Analysis/CMakeLists.txt
  clang/lib/Analysis/MacroExpansionContext.cpp
  clang/unittests/Analysis/CMakeLists.txt
  clang/unittests/Analysis/MacroExpansionContextTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93222.314857.patch
Type: text/x-patch
Size: 25987 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210106/088970cf/attachment-0001.bin>