[PATCH] D96097: [TargetLowering] Use Align in allowsMisalignedMemoryAccesses.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 4 19:22:41 PST 2021


craig.topper added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:1305
+                                    Align Alignment) {
+  if (Alignment.value() % SzInBytes == 0)
     return false;
----------------
arsenm wrote:
> I think this doesn't need the .value
I think I added it after getting a compile fail.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96097



More information about the llvm-commits mailing list