[PATCH] Refactor commoning of target specific load/store intrinsics in EarlyCSE

[hfinkel at anl.gov]

================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:467
@@ +466,3 @@
+  /// Must have one memory operand.
+  bool isTargetIntrinsicLikeLoad(const IntrinsicInst *II) const;
+
----------------
We have the IntrReadArgMem intrinsic property, how does this differ?


================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:498
@@ -492,1 +497,3 @@
+  /// \returns True if the target memory intrinsic may read memory.
+  bool mayTargetIntrinsicReadFromMemory(const IntrinsicInst *II) const;
   /// @}
----------------
II->mayReadFromMemory()?

http://reviews.llvm.org/D8313

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/