[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