Page MenuHomePhabricator

[WebAssembly][NFC] Replace WASM with standard Wasm
ClosedPublic

Authored by tlively on Aug 11 2020, 11:06 AM.

Details

Summary

The officially specified abbreviation for WebAssembly is Wasm and the
spec explicitly calls out WASM as being an incorrect spelling. This
patch fixes a few comments and error messages to use the
spec-compliant abbreviation.

Diff Detail

Unit TestsFailed

TimeTest
20 mslinux > Flang.Preprocessing::compiler_defined_macros.F90
Script: -- : 'RUN: at line 6'; /mnt/disks/ssd0/agent/llvm-project/build/bin/f18 -intrinsic-module-directory /mnt/disks/ssd0/agent/llvm-project/build/tools/flang/include/flang -E /mnt/disks/ssd0/agent/llvm-project/flang/test/Preprocessing/compiler_defined_macros.F90 | /mnt/disks/ssd0/agent/llvm-project/build/bin/FileCheck --ignore-case /mnt/disks/ssd0/agent/llvm-project/flang/test/Preprocessing/compiler_defined_macros.F90

Event Timeline

tlively created this revision.Aug 11 2020, 11:06 AM
tlively requested review of this revision.Aug 11 2020, 11:06 AM
aheejin accepted this revision.Aug 11 2020, 12:15 PM
This revision is now accepted and ready to land.Aug 11 2020, 12:15 PM
This revision was landed with ongoing or failed builds.Aug 11 2020, 12:28 PM
This revision was automatically updated to reflect the committed changes.
dschuff added inline comments.Aug 11 2020, 12:45 PM
llvm/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp
871

Is there any chance we can give more information here about which instruction or which feature is the offender?

tlively added inline comments.Aug 11 2020, 4:05 PM
llvm/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp
871