[PATCH] D32769: [X86][LWP] Add llvm support for LWP instructions.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 3 06:15:52 PDT 2017


RKSimon added a comment.

In https://reviews.llvm.org/D32769#744477, @andreadb wrote:

> I noticed that according to X86GenInstrInfo.inc, the new LWPINS instructions don't have "UnmodeledSideEffects".
>  However, all the other LWP instructions have flag `UnmodeledSideEffects` set.
>  My understanding is that this is because LWPVAL/LLWPCB/SLWPCB are directly matched from x86 intrinsics with side effects, while LWPINS* are firstly lowered to X86ISD::LWPINS nodes.
>
> Shouldn't we set `hasSideEffects = 1` for the LWPINS variants too? After all, those instructions modify memory by pushing new events in the ring buffer. So, those can both read and write.


I think I just need to add SDNPSideEffect to X86lwpins.


Repository:
  rL LLVM

https://reviews.llvm.org/D32769





More information about the llvm-commits mailing list