[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