[PATCH] D65048: [GISel]: Attach missing range metadata while translating G_LOADs
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jul 21 06:05:34 PDT 2019
arsenm added inline comments.
================
Comment at: llvm/lib/CodeGen/GlobalISel/IRTranslator.cpp:892
Ptr, Flags, (MRI->getType(Regs[i]).getSizeInBits() + 7) / 8,
- MinAlign(BaseAlign, Offsets[i] / 8), AAMDNodes(), nullptr,
+ MinAlign(BaseAlign, Offsets[i] / 8), AAMDNodes(), Ranges,
LI.getSyncScopeID(), LI.getOrdering());
----------------
How does this work if it's split into multiple register? I would assume this needs to be decomposed and only trivially valid if there is one result register
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65048/new/
https://reviews.llvm.org/D65048
More information about the llvm-commits
mailing list