Bug 227010 - devel/llvm60: Possible clang60 regression in complex numbers handling
Summary: devel/llvm60: Possible clang60 regression in complex numbers handling
Status: Closed Works As Intended
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Brooks Davis
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-03-27 18:29 UTC by Yuri Victorovich
Modified: 2019-01-17 18:52 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (brooks)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Yuri Victorovich freebsd_committer 2018-03-27 18:29:00 UTC
Here is the clang bug report with the testcase: https://bugs.llvm.org/show_bug.cgi?id=36915

This affects the port audio/x42-plugins-lv2 on 12 that has clang60.

The error message:
In file included from src/tuna.c:70:
src/spectr.c:136:32: error: implicit conversion from '_Complex int' to 'double' is not permitted in C++
                complex_t p = cos (omega) +  _I * sin (omega);
                                             ^~
Comment 1 Brooks Davis freebsd_committer 2019-01-17 18:52:14 UTC
Upstream seems to think that this is programmer error and that what changed was that clang generated an error instead of miscompiling the code.  Given a lack of progress on the suggested compatibility support upstream, I'm closing this for now and assuming that we'll get improvements (if any) in a future clang/llvm release.