[PATCH] D71238: Align non-fused branches within 32-Byte boundary (basic case)

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 20:39:23 PST 2019


skan added inline comments.


================
Comment at: llvm/lib/MC/MCAssembler.cpp:1007-1008
+
+///  getPaddingSize - Get how many bytes need to be padded to align branch with
+///  given address if the branch cross or is against the boundary.
+static unsigned getPaddingSize(unsigned StartAddr, unsigned BoundarySize) {
----------------
It seems that you didn't simply the code based on my latest patch, I have removed the class/function name in my comment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71238





More information about the llvm-commits mailing list