Message ID | 1458654386-1001-1-git-send-email-kbastian@mail.uni-paderborn.de |
---|---|
State | New |
Headers | show |
On 22 March 2016 at 13:46, Bastian Koppelmann <kbastian@mail.uni-paderborn.de> wrote: > The following changes since commit 4829e0378dfb91d55af9dfd741bd09e8f2c4f91a: > > Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2016-03-18' into staging (2016-03-18 17:18:41 +0000) > > are available in the git repository at: > > https://github.com/bkoppelmann/qemu-tricore-upstream.git tags/pull-tricore-20160322 > > for you to fetch changes up to d66718ccf37b71eff686c3d6116223889736188f: > > target-tricore: Add ftoi and itof instructions (2016-03-21 18:04:16 +0100) Hi; I'm afraid this doesn't build with clang: /home/petmay01/linaro/qemu-for-merges/target-tricore/fpu_helper.c:45:20: error: unused function 'f_is_pos_inf' [-Werror,-Wunused-function] static inline bool f_is_pos_inf(float32 arg) ^ /home/petmay01/linaro/qemu-for-merges/target-tricore/fpu_helper.c:50:20: error: unused function 'f_is_neg_inf' [-Werror,-Wunused-function] static inline bool f_is_neg_inf(float32 arg) ^ (clang is stricter than gcc about unused static functions in .c files.) thanks -- PMM
On 03/22/2016 09:15 PM, Peter Maydell wrote: > On 22 March 2016 at 13:46, Bastian Koppelmann > <kbastian@mail.uni-paderborn.de> wrote: >> The following changes since commit 4829e0378dfb91d55af9dfd741bd09e8f2c4f91a: >> >> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2016-03-18' into staging (2016-03-18 17:18:41 +0000) >> >> are available in the git repository at: >> >> https://github.com/bkoppelmann/qemu-tricore-upstream.git tags/pull-tricore-20160322 >> >> for you to fetch changes up to d66718ccf37b71eff686c3d6116223889736188f: >> >> target-tricore: Add ftoi and itof instructions (2016-03-21 18:04:16 +0100) > > Hi; I'm afraid this doesn't build with clang: > > /home/petmay01/linaro/qemu-for-merges/target-tricore/fpu_helper.c:45:20: > error: unused function 'f_is_pos_inf' [-Werror,-Wunused-function] > static inline bool f_is_pos_inf(float32 arg) > ^ > /home/petmay01/linaro/qemu-for-merges/target-tricore/fpu_helper.c:50:20: > error: unused function 'f_is_neg_inf' [-Werror,-Wunused-function] > static inline bool f_is_neg_inf(float32 arg) > ^ Whoops, these functions are not supposed to be there. I'll remove them and re-spin. Cheers, Bastian