Проверку на null перенести в начало метода. Оператор |= поменять на оператор = Добавить flags=0 после первого if, так как нет информации как работает метод trygetvalue

а если не зайдём ни в один if - получается попытка вернуть необъявленный символ. это законно в C#? upd: хотя кажется var out это и есть объявление, а туда безусловно зайдём.

а перед обращением к полям resolvedType не нужно проверять на null и бросать исключение например?

я не шарю за C# но меня смущает что флаг только устанавливается. не хватает сброса. что-то вроде flags &= ~HierarchyFlags.IsSystemReflectionIReflect; по ветке else (не вижу инициализацию flags нулём)

А вообще я слышал что хорошим тоном считается для наборов флагов использовать битовые поля (если нет в языке - не должно быть большой проблемой сделать такой модуль самостоятельно)

Out var ?