[PATCH] D16549: [MC] Use .p2align instead of .align

Derek Schuff via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 25 12:25:11 PST 2016


dschuff added a comment.

In general I like this a lot. But I fear there is probably some obscure assembler on some target somewhere that does not support .p2align. Maybe we need some kind of target hook to allow use of .align (either alongside getAlignmentIsInBytes() or perhaps replacing or subsuming it)? Or we could just propose/do this and let someone complain with a specific use case before doing that?


Repository:
  rL LLVM

http://reviews.llvm.org/D16549





More information about the llvm-commits mailing list