[PATCH] D63633: [WebAssembly] Fix p2align in assembler.

Wouter van Oortmerssen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 20 16:56:59 PDT 2019


aardappel created this revision.
aardappel added reviewers: aheejin, sbc100.
Herald added subscribers: llvm-commits, jfb, sunfish, jgravelle-google.
Herald added a project: LLVM.

- Match the syntax output by InstPrinter.
- Fix it always emitting 0 for align. Had to work around fact that opcode is not available for GetDefaultP2Align while parsing.
- Updated tests that were erroneously happy with a p2align=0

Fixes https://bugs.llvm.org/show_bug.cgi?id=40752


Repository:
  rL LLVM

https://reviews.llvm.org/D63633

Files:
  lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp
  lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCTargetDesc.h
  test/MC/WebAssembly/atomics-encodings.s
  test/MC/WebAssembly/basic-assembly.s
  test/MC/WebAssembly/reloc-pic.s
  test/MC/WebAssembly/simd-encodings.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63633.205925.patch
Type: text/x-patch
Size: 16166 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190620/b26b4067/attachment.bin>


More information about the llvm-commits mailing list