[PATCH] D72738: [BranchAlign] Add master --x86-branches-within-32B-boundaries flag

[Kan Shengchen via Phabricator via llvm-commits]

skan added a comment.

In D72738#1820811 <https://reviews.llvm.org/D72738#1820811>, @MaskRay wrote:

> I think the most recent review for that is D72463 <https://reviews.llvm.org/D72463>. D72227 <https://reviews.llvm.org/D72227> just copied what I did in D72463 <https://reviews.llvm.org/D72463>.


Just to clarify that D72227 <https://reviews.llvm.org/D72227> was opened first, then D72463 <https://reviews.llvm.org/D72463> was opened later and changed the principle of option override on the basis of D72227 <https://reviews.llvm.org/D72227>'s code and made some refinements. After that, some refinements were migrated back to D72227 <https://reviews.llvm.org/D72227>. D72227 <https://reviews.llvm.org/D72227> and D72463 <https://reviews.llvm.org/D72463> always adhere to different override principle. It is inappropriate to say who copied it.


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

https://reviews.llvm.org/D72738