[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