[PATCH] D57499: [WebAssembly] clang-tidy (NFC)

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 30 22:59:16 PST 2019


aheejin created this revision.
aheejin added a reviewer: sbc100.
Herald added subscribers: llvm-commits, sunfish, jgravelle-google, dschuff.

This patch fixes clang-tidy warnings on wasm-only files.
The list of checks used is:
`-*,clang-diagnostic-*,llvm-*,misc-*,-misc-unused-parameters,readability-identifier-naming,modernize-*`
(LLVM's default .clang-tidy list is the same except it does not have
`modernize-*`.)

The list of fixes are:

- Variable names start with an uppercase letter
- Function names start with a lowercase letter
- Use `auto` when you use casts so the type is evident
- Replaces explicit calls to the constructor in a return with a braced

initializer list


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D57499

Files:
  wasm/InputChunks.cpp
  wasm/MarkLive.cpp
  wasm/Writer.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57499.184456.patch
Type: text/x-patch
Size: 4878 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190131/cd62eef7/attachment.bin>


More information about the llvm-commits mailing list