llvm-project
832285ab - [clang][NFC] Unify `MacroState` `isAmbiguous` and `getModuleInfo` (#197867)

Commit
100 days ago
[clang][NFC] Unify `MacroState` `isAmbiguous` and `getModuleInfo` (#197867) Every call to `MacroState::getModuleInfo`, and `MacroState::isAmbiguous` are paired in the same function. Rather than doing the same work twice, just unify them into a single function, `getModuleInfo`, that returns both pieces of information in a new type `ModuleMacroInfo`. Unfortunately, `getModuleInfo` and`ModuleMacroInfo` already exist, so rename them to `getFullModuleInfo` and `FullModuleMacroInfo`, respectively, since the new type is a subset of the old type. The new type contains just the pieces consumers care about. While we're there, use the range constructor of `llvm::DenseSet` instead of default constructing and calling `insert` in a loop.
Author
Parents
Loading