nickdesaulniers marked 3 inline comments as done. nickdesaulniers added inline comments.
================ Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:1954 + // than 23760. + // It might be nice to use AArch64::MOVi32imm here, which would get + // expanded in PreSched2 after PostRA, but our lone scratch Reg already ---------------- dmgreen wrote: > It may be possible to do something earlier, where a we add the MOVi32imm > earlier before registry allocation and use that as the register for the ldr > offset. That would keep it simpler than trying to find scratch registers, and > may be optimized better by the pipeline. Sure; added that as a comment; hopefully we wont need it, but "never say never (again)." Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100919/new/ https://reviews.llvm.org/D100919 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
