[PATCH] D97982: [MC] Introduce NeverAlign fragment type

Amir Ayupov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 6 16:04:48 PDT 2021


Amir added a comment.

@MaskRay: thank you for getting back to it. I understand the review process, but without an actionable feedback it's hard to move forward. I also understand the fatigue of this back and forth for reviewers. Thus I appreciate your getting back to this diff.
Agree that this solution may appear complicated and redundant wrt other (existing, simpler) mechanisms. However, the drawbacks of other solutions were discussed here and are outlined in the summary. Honest question: what other collateral do you think could show it's the right design?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97982



More information about the llvm-commits mailing list