cleaned up FwdAnalysis

[firewave]

No description provided.

[danmar]

Maybe I missed it. But I feel these should be in some namespace to avoid possible ODR violations.

Suggested change

	struct Result {
	enum class Type : std::uint8_t { NONE, READ, WRITE, BREAK, RETURN, BAILOUT } type;
	explicit Result(Type type) : type(type) {}
	Result(Type type, const Token *token) : type(type), token(token) {}
	const Token* token{};
	};
	enum class What : std::uint8_t { Reassign, UnusedValue };
	namespace {
	struct Result {
	enum class Type : std::uint8_t { NONE, READ, WRITE, BREAK, RETURN, BAILOUT } type;
	explicit Result(Type type) : type(type) {}
	Result(Type type, const Token *token) : type(type), token(token) {}
	const Token* token{};
	};
	enum class What : std::uint8_t { Reassign, UnusedValue };
	}

[firewave]

I am not sure that is necessary as clang-tidy should complain about that.