Add support for the intrinsic llvm.ppc.eieio to emit the instruction eieio.
rzurob stefanp hfinkel nemanjai
- Group Reviewers
- rGab4d60642142: [PowerPC] Add support for intrinsic llvm.ppc.eieio
This used to be under PowerPC Instructions used for assembler/disassembler only. Is this no longer true? You seem to have moved this into PPC32 Store Instructions section. What does this intrinsic do?
We generally don't move unrelated code around. Maybe keep this where it was and add your new def under PowerPC Assembler Instruction Aliases?
We are exposing it as an intrinsic, so I think it is not just for assembler/disassembler only so should not be in the previous place anymore.
It is a form of barrier that acts on only load and stores related to data not instructions.
The purpose of this patch is to expose this instruction using a built in function. You are correct that using the hasSideEffects bit will make it clear, but I checked that there are a lot of synchronization instructions without the hasSideEffects bit set, I think all of them should be handled together in a separate patch.