[PATCH] D122065: [BOLT] Align constant islands to 8 bytes

[Vladislav Khmelevsky via Phabricator via llvm-commits]

yota9 marked an inline comment as done.
yota9 added a comment.

Thank you for your review @maksfb ! I agree with your concerns but currently I don't have a good idea too. Within the time I hope we will find a solution :)



================
Comment at: bolt/test/AArch64/constant-island-alignment.s:12
+
+# CHECK: {{0|8}} <$d>:
+
----------------
maksfb wrote:
> nit: place the CHECK next to the asm directive that corresponds to the island being checked. I don't know how the names are assigned for islands and if it's possible to give it a distinctive name.
Done. The $d is the arm ABI name, there is no way to manipulate this AFAIK 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122065