[clangd][feature request] add hover comments for import statements

[mccakit]

Currently hovering over import statements don't have hovering comments

If you have a module like so:

///some comment
export module math;

You can see anything if you hover over this

import math;

Adding this feature would make it easier for end users of the libraries to know what module does what.

[YexuanXiao]

Anything? At least it doesn't apply to the std module.

[YexuanXiao]

Perhaps it can display all the exported namespaces instead of anything.

[llvmbot]

@llvm/issue-subscribers-clangd

Author: mccakit (mccakit)

Currently hovering over import statements don't have hovering comments

If you have a module like so:

///some comment
export module math;

You can see anything if you hover over this

import math;

Adding this feature would make it easier for end users of the libraries to know what module does what.

[llvmbot]

@llvm/issue-subscribers-clang-modules

Author: mccakit (mccakit)

Currently hovering over import statements don't have hovering comments

If you have a module like so:

///some comment
export module math;

You can see anything if you hover over this

import math;

Adding this feature would make it easier for end users of the libraries to know what module does what.

[HighCommander4]

To clarify, what you'd like to appear in the hover is the comment ///some comment that appeared above export module math; in the module interface file?

[mccakit]

Yes, that is correct

[ChuanqiXu9]

This is slightly harder than other feature request since we need a module declaration AST node in clang to support this. But we don't have it right now.

BTW, except this, I feel but I didn't count, many other feature request to modules support in clangd are actually easier than most people think. All most logics live in ModulesBuilder.cpp and ScanningProjectModules.cpp. I feel a lot of programmers are able to take a look.