[PATCH] D67229: [Alignment][NFC] Use Align with TargetLowering::setMinFunctionAlignment

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 5 09:19:57 PDT 2019


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


================
Comment at: llvm/lib/Target/SystemZ/SystemZISelLowering.cpp:123
   // Instructions are strings of 2-byte aligned 2-byte values.
-  setMinFunctionLogAlignment(2);
+  setMinFunctionAlignment(llvm::Align(4));
   // For performance reasons we prefer 16-byte alignment.
----------------
This one seems suspicious, it looks like it should be 2 byte aligned.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67229





More information about the llvm-commits mailing list