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

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 24 13:10:29 PDT 2019


aheejin added a comment.

Thanks!



================
Comment at: lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp:353
                        InstName.startswith("store") ||
                        InstName.startswith("atomic");
     if (IsLoadStore) {
----------------
Can atomic operations take `p2align`? Looks like alignments for atomic instructions are [[ https://github.com/WebAssembly/threads/blob/master/proposals/threads/Overview.md#spec-changes | predefined ]]. 


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63633/new/

https://reviews.llvm.org/D63633





More information about the llvm-commits mailing list