Re: [PATCH] Makes softmmu compile with clang at HEAD

2020-03-30 Thread Alex Bennée
Shu-Chun Weng writes: > With clang at HEAD, linking fails with "undefined symbol: > qemu_build_not_reached". It's because `store_helper` and > `helper_ret_stb_mmu` are mutually recursive and clang inlined latter > inside the former, making `store_helper` a recursive function and no > longer ful

Re: [PATCH] Makes softmmu compile with clang at HEAD

2020-03-30 Thread Shu-Chun Weng
Ping. On Tue, Mar 17, 2020 at 11:21 PM Shu-Chun Weng wrote: > With clang at HEAD, linking fails with "undefined symbol: > qemu_build_not_reached". It's because `store_helper` and > `helper_ret_stb_mmu` are mutually recursive and clang inlined latter > inside the former, making `store_helper` a r

[PATCH] Makes softmmu compile with clang at HEAD

2020-03-17 Thread Shu-Chun Weng
With clang at HEAD, linking fails with "undefined symbol: qemu_build_not_reached". It's because `store_helper` and `helper_ret_stb_mmu` are mutually recursive and clang inlined latter inside the former, making `store_helper` a recursive function and no longer fully inlineable preventing constant pr