[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?
More information about the llvm-commits