[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 mailing list