[PATCH] D57500: [WebAssembly] clang-tidy (NFC)
Heejin Ahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 30 22:59:17 PST 2019
aheejin created this revision.
aheejin added reviewers: dschuff, sunfish, sbc100, jgravelle-google, tlively, aardappel, yurydelendik, kripken.
Herald added subscribers: llvm-commits, rupprecht, mgorny, MatzeB.
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-*`. But I've seen in multiple CLs in LLVM the modernize style
was recommended and code was fixed based on the style, so I added it as
well.)
The common 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
- Use inline initialization for class member variables
- Use `= default` for unnecessary default constructors / destructors
- Replaces explicit calls to the constructor in a return with a braced
initializer list
Repository:
rL LLVM
https://reviews.llvm.org/D57500
Files:
include/llvm/MC/MCSectionWasm.h
include/llvm/MC/MCWasmStreamer.h
lib/BinaryFormat/Wasm.cpp
lib/MC/MCParser/WasmAsmParser.cpp
lib/MC/MCSectionWasm.cpp
lib/MC/MCWasmObjectTargetWriter.cpp
lib/MC/MCWasmStreamer.cpp
lib/MC/WasmObjectWriter.cpp
lib/Object/WasmObjectFile.cpp
lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp
lib/Target/WebAssembly/CMakeLists.txt
lib/Target/WebAssembly/InstPrinter/WebAssemblyInstPrinter.cpp
lib/Target/WebAssembly/MCTargetDesc/WebAssemblyAsmBackend.cpp
lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.cpp
lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCAsmInfo.h
lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCCodeEmitter.cpp
lib/Target/WebAssembly/MCTargetDesc/WebAssemblyMCTargetDesc.cpp
lib/Target/WebAssembly/MCTargetDesc/WebAssemblyWasmObjectWriter.cpp
lib/Target/WebAssembly/WebAssemblyAsmPrinter.cpp
lib/Target/WebAssembly/WebAssemblyAsmPrinter.h
lib/Target/WebAssembly/WebAssemblyCFGSort.cpp
lib/Target/WebAssembly/WebAssemblyCFGStackify.cpp
lib/Target/WebAssembly/WebAssemblyCallIndirectFixup.cpp
lib/Target/WebAssembly/WebAssemblyExplicitLocals.cpp
lib/Target/WebAssembly/WebAssemblyFastISel.cpp
lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp
lib/Target/WebAssembly/WebAssemblyISelDAGToDAG.cpp
lib/Target/WebAssembly/WebAssemblyISelLowering.cpp
lib/Target/WebAssembly/WebAssemblyISelLowering.h
lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp
lib/Target/WebAssembly/WebAssemblyLowerGlobalDtors.cpp
lib/Target/WebAssembly/WebAssemblyMCInstLower.cpp
lib/Target/WebAssembly/WebAssemblyMCInstLower.h
lib/Target/WebAssembly/WebAssemblyMachineFunctionInfo.cpp
lib/Target/WebAssembly/WebAssemblyMachineFunctionInfo.h
lib/Target/WebAssembly/WebAssemblyMemIntrinsicResults.cpp
lib/Target/WebAssembly/WebAssemblyOptimizeLiveIntervals.cpp
lib/Target/WebAssembly/WebAssemblyOptimizeReturned.cpp
lib/Target/WebAssembly/WebAssemblyPeephole.cpp
lib/Target/WebAssembly/WebAssemblyPrepareForLiveIntervals.cpp
lib/Target/WebAssembly/WebAssemblyRegColoring.cpp
lib/Target/WebAssembly/WebAssemblyRegStackify.cpp
lib/Target/WebAssembly/WebAssemblyRuntimeLibcallSignatures.cpp
lib/Target/WebAssembly/WebAssemblyRuntimeLibcallSignatures.h
lib/Target/WebAssembly/WebAssemblySelectionDAGInfo.cpp
lib/Target/WebAssembly/WebAssemblySelectionDAGInfo.h
lib/Target/WebAssembly/WebAssemblySetP2AlignOperands.cpp
lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp
lib/Target/WebAssembly/WebAssemblyTargetMachine.h
lib/Target/WebAssembly/WebAssemblyTargetTransformInfo.cpp
tools/llvm-objdump/WasmDump.cpp
tools/llvm-readobj/WasmDumper.cpp
tools/obj2yaml/wasm2yaml.cpp
tools/yaml2obj/yaml2wasm.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57500.184457.patch
Type: text/x-patch
Size: 163504 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190131/84e41ec1/attachment-0001.bin>
More information about the llvm-commits
mailing list