[PATCH] D36862: AMDGPU: Handle non-temporal loads and stores

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/SIMemoryLegalizer.cpp:115
+    MachineOperand &Bit = MI->getOperand(BitIdx);
+    if (Bit.getImm() == 1)
+      return false;
----------------
It would be better to check if != 0. If you define an operand as an i1imm for example it is the sign extended constant value -1


https://reviews.llvm.org/D36862