<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<font size="2"><span style="font-size:11pt">> For some of us, performance *is* correctness. :)<br>
</span></font></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<font size="2"><span style="font-size:11pt">><br>
> IMO this option should emit a warning at least.</span></font></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
I obviously agree with that, but I don't see the benefit when that is emitted by using an opt-in flag, which was the consensus if I'm not mistaken.</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
So the absolute minimum we can do is to document this behaviour, which is what I did with
<span>rG35cf2f42dda4</span>.<br>
</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> David Greene <david.greene@hpe.com><br>
<b>Sent:</b> 22 April 2020 16:59<br>
<b>To:</b> David Blaikie <dblaikie@gmail.com>; Sjoerd Meijer <Sjoerd.Meijer@arm.com><br>
<b>Cc:</b> cfe-dev@lists.llvm.org Developers <cfe-dev@lists.llvm.org><br>
<b>Subject:</b> Re: [cfe-dev] Option -mtune</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">David Blaikie via cfe-dev <cfe-dev@lists.llvm.org> writes:<br>
<br>
>> Thanks for your reply, and I can see that this is the only benefit of this<br>
>> no-op: if your move to Clang and if you still have -mtune set in your build<br>
>> environment, you don't get an "unknown option" error. But for me<br>
>> personally, silently ignoring things and giving the impression it works is<br>
>> worse than a simple Makefile fix, but maybe I am wrong, which is why I<br>
>> checked here on the list.<br>
>><br>
><br>
> If it were a matter of correctness rather than performance, I'd agree -<br>
<br>
For some of us, performance *is* correctness. :)<br>
<br>
IMO this option should emit a warning at least.<br>
<br>
-David<br>
</div>
</span></font></div>
</body>
</html>