[PATCH] D98304: [AMDGPU] Create AMDGPUModuleInfo

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 11 00:01:51 PST 2021


rampitec added a comment.

Ugh, that's the problem: I cannot tell if a context is the same as an old one. I can tell it for module as I can create a module pass. Every time module changes I can reset the context. I cannot create a context pass though.

Now I cannot compare context address as well since the same memory can be reused. This is exactly the same problem as with a module, its address can be the same, but the module different. I.e. there is no reliable way to identify a module, but there is a module pass, and there is no reliable way to identify a context, but there are no context passes.

Effectively even though this is a context extension I need to reset it for every new module. Well, unless you tell me there is a way to reliably identify a context.


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

https://reviews.llvm.org/D98304



More information about the llvm-commits mailing list