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

[Sam Parker via Phabricator via llvm-commits]

samparker added a comment.

> We're moving to a world where we want the alignment to be explicit at the IR level for loads and stores

Ah, thanks for the update, it sounds like a good direction.

> are there any memory operations handled by getMemoryOpCost that are neither load nor store?

As far as I can tell, no... but I also see little enforcement. The masked intrinsics have their own API but I guess there's nothing stopping a backend from using this API to get a baseline for any of it's 'memory' operations. I will update this patch to move to Align instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80924