search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

build of clang apply replacement is failing with an "invalid conversion" #32942

Closed Quuxplusone closed 7 years ago

Quuxplusone commented 7 years ago
Bugzilla Link PR33970
Status RESOLVED FIXED
Importance P enhancement
Reported by Sylvestre Ledru (sylvestre@debian.org)
Reported on 2017-07-27 23:42:24 -0700
Last modified on 2017-07-31 23:55:20 -0700
Version unspecified
Hardware PC Linux
CC bero@linaro.org, klimek@google.com
Fixed by commit(s)
Attachments clang-apply-replacements-compilefix.patch (647 bytes, text/plain)
Blocks PR33849
Blocked by
See also 
with 5.0 svn branch 309348

/build/llvm-toolchain-5.0-5.0~svn309348/tools/clang/tools/extra/clang-apply-
replacements/tool/ClangApplyReplacementsMain.cpp: In function 'int main(int,
char**)':
/build/llvm-toolchain-5.0-5.0~svn309348/tools/clang/tools/extra/clang-apply-
replacements/tool/ClangApplyReplacementsMain.cpp:202:37: error: invalid
conversion from 'void (*)(llvm::raw_ostream&)' to 'void (*)()' [-fpermissive]
   cl::SetVersionPrinter(printVersion);
                                     ^
In file included from /build/llvm-toolchain-5.0-
5.0~svn309348/tools/clang/include/clang/Frontend/CommandLineSourceLoc.h:19:0,
                 from /build/llvm-toolchain-5.0-5.0~svn309348/tools/clang/include/clang/Frontend/FrontendOptions.h:13,
                 from /build/llvm-toolchain-5.0-5.0~svn309348/tools/clang/include/clang/Frontend/FrontendAction.h:25,
                 from /build/llvm-toolchain-5.0-5.0~svn309348/tools/clang/include/clang/Tooling/Tooling.h:39,
                 from /build/llvm-toolchain-5.0-5.0~svn309348/tools/clang/include/clang/Tooling/Refactoring.h:23,
                 from /build/llvm-toolchain-5.0-5.0~svn309348/tools/clang/tools/extra/clang-apply-replacements/include/clang-apply-replacements/Tooling/ApplyReplacements.h:20,
                 from /build/llvm-toolchain-5.0-5.0~svn309348/tools/clang/tools/extra/clang-apply-replacements/tool/ClangApplyReplacementsMain.cpp:16:
/build/llvm-toolchain-5.0-
5.0~svn309348/include/llvm/Support/CommandLine.h:74:6: note:   initializing
argument 1 of 'void llvm::cl::SetVersionPrinter(void (*)())'
 void SetVersionPrinter(void (*func)());
      ^~~~~~~~~~~~~~~~~
Quuxplusone commented 7 years ago

with gcc 6 & 4.9

Quuxplusone commented 7 years ago

Attached clang-apply-replacements-compilefix.patch (647 bytes, text/plain): Fix

Quuxplusone commented 7 years ago

Reverted in r304837 by hans