[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: Re: [OpenJDK 2D-Dev] RFR JDK-8209786: gcc 7.3 compiler errors on zLinux
From: Brian Burkhalter <brian.burkhalter () oracle ! com>
Date: 2018-08-31 14:43:29
Message-ID: 7BF23440-41CE-43DB-AA68-BE250D286C1E () oracle ! com
[Download RAW message or body]
On Aug 31, 2018, at 2:28 AM, Magnus Ihse Bursie <magnus.ihse.bursie@oracle.com> \
wrote:
> > Magnus, Philip, Brian, Goetz, can we have a vote? => "Fix" or "DisableWarnings" ?
>
> Note that this decision can be different for the two libraries. I'd argue that the \
> maintainer of each library decides. And if so, it seems to be "compiler fix" for \
> libfdlibm, and "source fix" for libmlib_image.
I think we can safely say “disable compiler errors” for fdlibm as indicated by Joe \
Darcy’s comment in the issue (he owns fdlibm), and source code change for mediaLib as \
Phil indicated in e-mail.
Thanks,
Brian
[Attachment #3 (unknown)]
<html><head><meta http-equiv="Content-Type" content="text/html \
charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: \
space; -webkit-line-break: after-white-space;"><br><div><div>On Aug 31, 2018, at 2:28 \
AM, Magnus Ihse Bursie <<a \
href="mailto:magnus.ihse.bursie@oracle.com">magnus.ihse.bursie@oracle.com</a>> \
wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div \
text="#000000" bgcolor="#FFFFFF"><blockquote type="cite" \
cite="mid:OF2A92E894.CE374AFC-ON802582FA.00317DC3-802582FA.0032C01A@notes.na.collabserv.com" \
style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: \
normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: \
auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; \
widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><span \
style="font-size: 12pt;">Magnus, Philip, Brian, Goetz, can we have a vote? => \
"Fix" or "DisableWarnings" ?</span></blockquote><br style="font-family: Helvetica; \
font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; \
letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; \
text-indent: 0px; text-transform: none; white-space: normal; widows: auto; \
word-spacing: 0px; -webkit-text-stroke-width: 0px;"><span style="font-family: \
Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: \
normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: \
start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; \
word-spacing: 0px; -webkit-text-stroke-width: 0px; background-color: rgb(255, 255, \
255); float: none; display: inline !important;">Note that this decision can be \
different for the two libraries. I'd argue that the maintainer of each library \
decides. And if so, it seems to be "compiler fix" for libfdlibm, and "source fix" for \
libmlib_image.</span></div></blockquote></div><br><div>I think we can safely say \
“disable compiler errors” for fdlibm as indicated by Joe Darcy’s comment in the issue \
(he owns fdlibm), and source code change for mediaLib as Phil indicated in \
e-mail.</div><div><br></div><div>Thanks,</div><div><br></div><div>Brian</div></body></html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic