[PATCH] D74995: [AMDGPU] Revert 'Don’t marke the .note section as ALLOC'

[Tony Tye via Phabricator via llvm-commits]

t-tye added a comment.

The NOTE section is currently being used to hold the high level runtime metadata, and that is only being read by the language runtimes and so does not need to be part of the loaded code object. So I do not think this note section should ever be marked as ALLOC.

I suspect the VDI runtime may be trying to determine how much device memory is available by subtracting the size of the code objects it will load. But it seems it would be better to get the information directly from the runtime memory allocator rather than trying to deduce it indirectly. I think part of the reason it does that may be because currently it is deferring loading code objects until they are first used. I do not think that is the right thing to do either as it means asking for a symbol address of a code object that the application thinks is loaded may fail due to the deferred load failing. It would be better for applications not to load vast numbers of code objects that it will never use (which is why deferred loading was implemented).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74995