[PATCH] [Polly] Added arcanist linters and cleaned errors and warnings
Johannes Doerfert
doerfert at cs.uni-saarland.de
Thu Aug 14 15:44:01 PDT 2014
Hi simbuerg, grosser, sebpop,
Arcanist (arc) will now always run linters before uploading any new
commit to Phabricator. All errors/warnings (or their absence) will be
shown in the web interface together with a explanation by the commiter
(arcanist will ask the commiter if the build was not clean).
The linters include:
- clang-format
- spelling check
- permissions check (aka. chmod)
- filename check
- merge conflict marker check
Note, that their scope is sometimes limited (see .arclint for
details).
This commit also fixes all errors and warnings these linters reported,
namely:
- spelling mistakes and typos
- executable permissions for various text files
http://reviews.llvm.org/D4916
Files:
.arcconfig
.arclint
CMakeLists.txt
autoconf/configure.ac
autoconf/configure.bak
include/polly/Canonicalization.h
include/polly/Config/config.h.cmake
include/polly/Dependences.h
include/polly/ScopPass.h
include/polly/Support/GICHelper.h
include/polly/Support/SCEVValidator.h
include/polly/Support/ScopHelper.h
lib/Analysis/ScopInfo.cpp
lib/JSON/include/json/value.h
lib/JSON/include/json/writer.h
lib/Transform/CodePreparation.cpp
test/Cloog/CodeGen/20120403-RHS-type-mismatch.ll
test/Cloog/ambigous_schedule.ll
test/Isl/CodeGen/20120403-RHS-type-mismatch.ll
test/Makefile
test/ScopInfo/Alias-0.ll
test/ScopInfo/Alias-1.ll
test/ScopInfo/Alias-2.ll
test/ScopInfo/Alias-3.ll
test/ScopInfo/Alias-4.ll
test/ScopInfo/bug_2010_10_22.ll
test/ScopInfo/bug_2011_1_5.ll
test/ScopInfo/phi_not_grouped_at_top.ll
test/ScopInfo/phi_with_invoke_edge.ll
utils/check_format.sh
www/video-js/video.js
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4916.12527.patch
Type: text/x-patch
Size: 8605 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140814/3b87e939/attachment.bin>
More information about the llvm-commits
mailing list