[PATCH] D109707: [HIP] [AlwaysInliner] Disable AlwaysInliner to eliminate undefined symbols
Anshil Gandhi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 23 10:05:18 PDT 2021
gandhi21299 added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUResourceUsageAnalysis.cpp:65
- return cast<Function>(Op.getGlobal());
+ return dyn_cast<Function>(Op.getGlobal());
}
----------------
arsenm wrote:
> I think this is not the right place for this. If we can determine the callee function, we should have directly set it in the instruction during call lowering
Which file would that be in?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109707/new/
https://reviews.llvm.org/D109707
More information about the llvm-commits
mailing list