kmclaughlin planned changes to this revision. kmclaughlin added a comment. Thanks for the feedback on this patch, @efriedma & @sdesmalen! I think there is still value in adding a NonFaulting flag to MachineMemOperand so that we can benefit from legalisation, but as this is not a requirement for the ACLE I have created a new patch which implements the non-faulting load intrinsic explicitly: https://reviews.llvm.org/D71698 I will leave this patch in the 'plan changes' state so that it can be referred to in future discussions on the mailing list.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71556/new/ https://reviews.llvm.org/D71556 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits