[PATCH] D92139: [PowerPC] Add `hasSideEffects=0` for PLXVP and PSTXVP instructions definition

[Qing Shan Zhang via Phabricator via llvm-commits]

steven.zhang added a comment.

I think, this patch isn't needed any more as we have already specify the match patter for these instructions and tblgen will infer the sideeffects from the patter. See what we get from tblgen now(we didn't have the sideeffect bit):

  { 1191,       3,      1,      4,      39,     0|(1ULL<<MCID::MayLoad), 0x0ULL, nullptr, nullptr, OperandInfo158 },  // Inst #1191 = LXVP


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92139