search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

LLVM Werror/Wx builds fail after D64609/rL366344 due to deprecation warnings #41637

Closed Quuxplusone closed 5 years ago

Quuxplusone commented 5 years ago
Bugzilla Link PR42667
Status RESOLVED FIXED
Importance P enhancement
Reported by Simon Pilgrim (llvm-dev@redking.me.uk)
Reported on 2019-07-18 06:03:39 -0700
Last modified on 2019-07-19 06:58:39 -0700
Version trunk
Hardware PC Windows NT
CC 1101.debian@gmail.com, hans@chromium.org, lhames@gmail.com, llvm-bugs@lists.llvm.org, stefan.graenitz@gmail.com
Fixed by commit(s) rL366511
Attachments
Blocks PR42474
Blocked by
See also

I'm seeing this (MSVC) via OrcCBindingsStack.h

and the (gcc/clang) buildbots are seeing this in clang-interpreter/main.cpp

/home/buildslave/ps4-buildslave4/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/llvm.src/tools/clang/examples/clang-interpreter/main.cpp:75:9: warning: 'LegacyRTDyldObjectLinkingLayer' is deprecated: ORCv1 layers (layers with the 'Legacy' prefix) are deprecated. Please use ORCv2 (see docs/ORCv2.rst) [-Wdeprecated-declarations]

/home/buildslave/ps4-buildslave4/llvm-clang-lld-x86_64-scei-ps4-ubuntu-fast/llvm.src/tools/clang/examples/clang-interpreter/main.cpp:80:9: warning: 'LegacyIRCompileLayer' is deprecated: ORCv1 layers (layers with the 'Legacy' prefix) are deprecated. Please use the ORCv2 IRCompileLayer instead [-Wdeprecated-declarations]

Quuxplusone commented 5 years ago

I have updated the clang-interpreter example to use ORCv2 in r366511. That should remove these warnings.

Quuxplusone commented 5 years ago 
(In reply to Lang Hames from comment #1)
> I have updated the clang-interpreter example to use ORCv2 in r366511. That
> should remove these warnings.

Merged to 9.0 in r366556.
Quuxplusone commented 5 years ago

Thanks! Keeping this open until r366556 is merged into the 9.00 release branch

Quuxplusone commented 5 years ago 
(In reply to Simon Pilgrim from comment #3)
> Thanks! Keeping this open until r366556 is merged into the 9.00 release
> branch

But r366556 is the merge to the 9.0.0 release branch..?
Quuxplusone commented 5 years ago

My mistake - sorry about that!