<div dir="ltr"><div dir="ltr">On Wed, 25 Nov 2020 at 18:10, Johannes Doerfert <<a href="mailto:johannesdoerfert@gmail.com">johannesdoerfert@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> If you only add infrastructure to build Fortran programs inside SPEC, then<br>
> your change would be biased towards an external benchmark that is private<br>
> to some companies.<br>
<br>
That doesn't make any sense to me.<br>
Nobody suggested to change anything "inside SPEC".<br></blockquote><div><br></div><div>Good part of your reply assumes I meant what you say above. I didn't.</div><div><br></div><div>We're talking past each other. Let me try again.</div><div><br></div><div>As I said on my original reply, I'm very supportive of the initiative to add Fortran to the test suite. To add tests, benchmarks and openmp. This is very good news.</div><div><br></div><div>But the test-suite doesn't have a core ownership, a group that has a plan and implements all the parts of a bigger design goal. For many years we have tried to unify tests and benchmarks, Kristof did a great job rallying people around and so many other people contributed, but once it "works", people stop paying attention.</div><div><br></div><div>I just want to make sure that the overall support for Fortran in the test-suite is focused on building tests, benchmarks and other tools that are available upstream to all users.</div><div><br></div><div>If adding Fortran support on the existing SPEC scripts is orthogonal, then it shouldn't be part of this discussion. If it's not, then it shouldn't be the main driver for the rest of the infrastructure.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> Public build-bots will start building those tests and benchmarks (remember,<br>
> it's not just benchmarks in there), and you'll need some time to adjust<br>
> strategy until it all works across the board.<br>
<br>
Strategy: If you don't set it up to run Fortran codes, it won't.<br></blockquote><div><br></div><div>I'm going to take this as a tongue-in-cheek comment. The reducionism here isn't really helpful. </div><div><br></div><div>Fortran is just the language, but there are architectures and operating systems that need adjusting, too.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Fortran benchmark support in the LLVM Test Suite, and literally<br>
everything else mentioned in the initial RFC, is beneficial to the<br>
community. SPEC support is not something harmful.<br></blockquote><div><br></div><div>We definitely agree on that.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">How did you come to that conclusion after the initial RFC explicitly listed<br>
other benchmarks and apps we want to include in to the test suite?<br></blockquote><div><br></div><div>The original RFC was very clear. Your response was less so. </div><div><br></div><div>On my reply to the RFC, I said I worry that we're focusing on SPEC too early. I'd rather make sure it works upstream before adding SPEC to the mix. </div><div><br></div><div>The reason I tried to convey (and clearly failed) is that the test-suite isn't a robust and well designed infrastructure, but a patch-work from different approaches over the years, which seems to "work fine" with what we have.</div><div><br></div><div>I may have read that wrong, but it sounded to me as if you were defending the prioritisation of SPEC "and some micro benchmarks" over the rest of the proposal.</div><div><br></div><div>I think that's a mistake, because it risks being the main thing that gets added and then not much else comes later (priorities change, etc).</div><div><br></div><div>If my interpretation is wrong, I apologise and we can ignore our past exchange. I'm still very supportive of this RFC. :)</div><div><br></div><div>cheers,</div><div>--renato</div></div></div>