[PATCH] D25174: BranchRelaxation: Account for function alignment

[Hal Finkel via llvm-commits]

hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

You need to update the comment, but otherwise this LGTM.



> BranchRelaxation.cpp:51
>      /// specified, return the offset the successor block will get if it has
>      /// this alignment.
> +    unsigned postOffset(const MachineBasicBlock &MBB) const {

Please update the comment to say that it takes the MBB of the following block.

https://reviews.llvm.org/D25174