[PATCH] D47004: [X86] Use addAliasForDirective to support the .word directive

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 8 02:14:06 PDT 2018


RKSimon added a comment.

I'm not very familiar with this code but it looks OK and the same approach is used in AARCH64, Sparc and Hexagon.

X86AsmParser::ParseDirectiveWord is approximately the same as AsmParser::parseDirectiveValue although AsmParser has extra checks (is asm in valid section, error checks for parseMany call).

@craig.topper Any comments?


https://reviews.llvm.org/D47004





More information about the llvm-commits mailing list