[PATCH] D63075: [TargetLowering] Add MachineMemOperand::Flags to allowsMemoryAccess tests (PR42123)

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 10 06:12:43 PDT 2019


RKSimon marked an inline comment as done.
RKSimon added inline comments.


================
Comment at: include/llvm/CodeGen/TargetLowering.h:1434
+                     bool *Fast = nullptr) const;
 
   /// Returns the target specific optimal type for load and store operations as
----------------
arsenm wrote:
> Should probably add an overload that just takes a const MachineMemOperand&
Good idea!


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63075/new/

https://reviews.llvm.org/D63075





More information about the llvm-commits mailing list