[PATCH] D98304: [AMDGPU] Create AMDGPUModuleInfo
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 10 16:24:54 PST 2021
arsenm added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUModuleInfo.cpp:57
+
+ this->M = M;
+ LLVMContext &CTX = M->getContext();
----------------
Why does this need to store a module? The context may be used by multiple modules. It shouldn't matter since this just needs the context IDs
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98304/new/
https://reviews.llvm.org/D98304
More information about the llvm-commits
mailing list