[PATCH] D80924: [CostModel] Use MaybeAlign in getMemoryOpCost

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 2 02:46:00 PDT 2020


samparker added a comment.

Well, firstly I'd just like the interface to be consistent throughout the cost model and, since the IR doesn't force memory operations to have an alignment, I don't see why we'd try to enforce that here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80924





More information about the llvm-commits mailing list