[PATCH] D71391: [PowerPC] Modify the hasSideEffects of some VSX instructions from 1 to 0

Zhang Kang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 12 00:25:20 PST 2019


ZhangKang added a comment.

In D71391#1781086 <https://reviews.llvm.org/D71391#1781086>, @Jim wrote:

> It looks like not only this 6 instructions have to set hasSideEffects to 0.


Yes, in fact, I am checking the all instructions. The 6 VSX instructions will make the case failed. And I don't want to submit a huge patch, it's hard to review, so I submitted this patch. 
The other instructions fixed will in another patch.


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

https://reviews.llvm.org/D71391





More information about the llvm-commits mailing list