<div dir="ltr">Yeah. I didn't mean to actually suggest that the compiler emit a warning diagnostic when you pass the flag. I only meant that an appropriately named flag would be, itself, a warning sign against inappropriate use, simply by its name, and the user having to explicitly pass it.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 22, 2020 at 2:55 PM Chris Lattner via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Jan 21, 2020, at 5:00 AM, Alex Bradbury <<a href="mailto:asb@lowrisc.org" target="_blank">asb@lowrisc.org</a>> wrote:<br>
>> This all makes sense to me.<br>
> <br>
> That's correct, thanks for the feedback.<br>
> <br>
> I do like the idea from James of having the compiler always spit out a<br>
> note when enabling the experimental extension, warning of its<br>
> experimental nature. If we had such a warning and additionally<br>
> required a `-riscv-enable-experimental-extensions` or similar, then I<br>
> think there could be merit in including in the ISA string as Simon<br>
> suggests, especially as we're likely to start putting that string in<br>
> ELF output etc.<br>
<br>
Are you suggesting this behavior from Clang or from LLVM?  I think it would be a bad thing for LLVM to produce this warning: there isn’t a precedent for this, and it breaks the library-based design goals.  Having clang produce a warning could be done, but it would be very noisy (one warning for every .c file in a build) and I’m not sure how much value it provides.<br>
<br>
-Chris<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>