[Bug 58381] New: "musl: Use __builtin_rint if available" breaks clang builds (except aarch64)

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=58381

Bug ID: 58381
           Summary: "musl: Use __builtin_rint if available" breaks clang
                    builds (except aarch64)
           Product: Wine
           Version: 9.10
          Hardware: x86-64
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: msvcrt
          Assignee: [e-mail]
          Reporter: [e-mail]
                CC: [e-mail]
   Regression SHA1: ee6e83dca874491329c5b376c2f9563093e3c69f
      Distribution: ---

Since "musl: Use __builtin_rint if available",
ee6e83dca874491329c5b376c2f9563093e3c69f, any calls to rint end up in infinite
loops, when built with Clang (except on aarch64).

If Clang doesn't have any good implementation of "__builtin_rint", it generates
a function call to "rint", even if built with -fno-builtin. See
https://gcc.godbolt.org/z/aYb3c1Ks1 for examples of the generated code. Only on
aarch64, Clang generates an inline implementation.

[Bug 58381] "musl: Use __builtin_rint if available" breaks clang builds (except aarch64)

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=58381

Bernhard Übelacker  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #1 from Bernhard Übelacker  ---
Hello, is wine version 9.10 really intended?

[Bug 58381] "musl: Use __builtin_rint if available" breaks clang builds (except aarch64)

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=58381

Martin Storsjö  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|9.10                        |unspecified

--- Comment #2 from Martin Storsjö  ---
(In reply to Bernhard Übelacker from comment #1)
> Hello, is wine version 9.10 really intended?

Oops, no, it definitely isn't - this is in the latest git past any tagged
release.

[Bug 58381] "musl: Use __builtin_rint if available" breaks clang builds (except aarch64)

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=58381

Ken Sharp  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|unspecified                 |10.10
           Keywords|                            |source

[Bug 58381] "musl: Use __builtin_rint if available" breaks clang builds (except aarch64)

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=58381

--- Comment #3 from Bernhard Übelacker  ---
I found with Krita an application that gets hit by this
with a llvm-newwow build, while trying to investigate bug #58381.

[Bug 58381] "musl: Use __builtin_rint if available" breaks clang builds (except aarch64)

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=58381

Piotr Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
         Resolution|---                         |FIXED
             Status|UNCONFIRMED                 |RESOLVED
      Fixed by SHA1|                            |02f3e6ea8a67d95b800d39e32d7
                   |                            |9a625561cd7e1

--- Comment #4 from Piotr Caban  ---
Marking as fixed by 02f3e6ea8a67d95b800d39e32d79a625561cd7e1.

[Bug 58381] "musl: Use __builtin_rint if available" breaks clang builds (except aarch64)

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=58381

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #5 from Alexandre Julliard  ---
Closing bugs fixed in 10.11.