[PATCH] D12067: AMDGPU: Refactor exp instructions

[Matt Arsenault via llvm-commits]

arsenm added a comment.

In https://reviews.llvm.org/D12067#486407, @axeldavy wrote:

> In https://reviews.llvm.org/D12067#484345, @arsenm wrote:
>
> > Update for trunk. Set mayLoad = 1 for EXP_DONE instead of hasSideEffects
>
>
> Could you explain why ? To me it looks like hasSideEffects is the only way to guarantee the exp with the done bit is last.
>
> Btw you forgot to update the comments in the part "Export opcodes" to reflect that change.


If exp_done is reading from unknown memory, it can't be reordered before the stores to unknown memory


https://reviews.llvm.org/D12067