[PATCH] D122065: [BOLT] Align constant islands to 8 bytes
Vladislav Khmelevsky via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 22 08:43:12 PDT 2022
yota9 marked an inline comment as done.
yota9 added a subscriber: rafaelauler.
yota9 added a comment.
Hello @t.p.northover ! I would we thankful I you will review the patch while the @rafaelauler is on vacation. The very main part is located in BinaryEmitter.cpp, that is we will always emit constant island on 8 bytes alignment since currently we don't track here in BOLT is the instruction that will load data from constant island may support unaligned access or not - we will just always aligne it on 8 bytes boundaries. Thank you for your time in advance!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122065/new/
https://reviews.llvm.org/D122065
More information about the llvm-commits
mailing list