rjmccall added inline comments.
================ Comment at: clang/lib/CodeGen/CGObjC.cpp:2351 - // FIXME: Do this when the target isn't aarch64. + llvm::Triple::ArchType Arch = CGF.CGM.getTriple().getArch(); + ---------------- Please leave a comment saying that we should do this on all targets, and the limiting factor is that it needs specific support from the backend. Otherwise, this patch LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111331/new/ https://reviews.llvm.org/D111331 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
