[PATCH] D71390: [PowerPC] Modify the hasSideEffects of MTLR and MFLR from 1 to 0

Zhang Kang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 17 17:38:53 PST 2019


ZhangKang added a comment.

In D71390#1788249 <https://reviews.llvm.org/D71390#1788249>, @nemanjai wrote:

> Something doesn't match up here. The description/title say that you are changing the flag for both `MFLR/MTLR`. But in the code I only see the change for `MTLR`. Why is that?
>  Also, the context is missing from this patch so it cannot be reviewed properly, please fix that.


Have updated the patch to add the context.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71390/new/

https://reviews.llvm.org/D71390





More information about the llvm-commits mailing list