Mark spill register as used so we will not choose it as offset register,
Otherwise, spill register may possibly get killed even before spill
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/CodeGen/AMDGPU/frame-pointer-spill.mir | ||
---|---|---|
4 | Could use better name and comment what this is testing | |
6–45 | You don't need most of these fields, probably just the stack related ones in machineFunctionInfo and tracksRegLiveness | |
46 | Why do you need so many stack objects? Can you just add one artificial object that's big enough to force the huge frame? |
Looks like this patch and the patch https://reviews.llvm.org/D85772 fix the same issue. Me and Austin discussed about it, and decided to go ahead with https://reviews.llvm.org/D85772 instead of this patch. So, I will close this patch once https://reviews.llvm.org/D85772 is merged.
The fix is already taken care in the patch https://reviews.llvm.org/D85772, hence, I am closing this patch.
Could use better name and comment what this is testing