[PATCH] D67121: [TargetLowering] Make allowsMemoryAccess methode virtual.

Thomas Raoux via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 9 18:15:22 PDT 2019


ThomasRaoux added a comment.

I added a new function allowsMemoryAccessForAlignment to handle the behavior of the previous allowsMemoryAccess function. I replaced the calls every except for DAGCombine as my understanding is that this place really mean to check that the memory access is legal and won't require legalization to expensive code sequence.

Is that what you had in mind?


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

https://reviews.llvm.org/D67121





More information about the llvm-commits mailing list