[PATCH] D50270: [x86] When using "and $0" and "orl $-1" to store 0 and -1 for minsize, make sure the store isn't volatile

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86InstrCompiler.td:1046
+  return !cast<StoreSDNode>(N)->isVolatile();
+}]>;
+
----------------
RKSimon wrote:
> Is there any place more generic we can put this? It doesn't seem x86 specific
It should probably be hoisted to include/llvm/Target/TargetSelectionDAG.td. SystemZ has this already with this name so we'd have to change that too. Do you want me move it and change SystemZ as a pre-patch. Or do you want it folded in here? Or move SystemZ as a followup?


https://reviews.llvm.org/D50270