<div dir="ltr">Hi Bernie!<div class="gmail_extra"><br></div><div class="gmail_extra">Thanks for the review!</div><div class="gmail_extra"><br></div><div class="gmail_extra">For a bit of context, my intention is not to fix any bug or behaviour, but to make the representation of cores more complete across all functions in Clang.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">There are a number of opportunities to fix stuff, de-duplicate functions, merge classes, and make it easier for people to use "-mcpu=cortex-a8" without defining "-target arm" and get all the defaults relevant. But that's for later, right now, I just want to get the names right. ;)</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On 6 September 2013 11:45, Bernie Ogden <span dir="ltr"><<a href="mailto:bogden@arm.com" target="_blank">bogden@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
1) That NEON test probably should be checking for startswith("8") as well as<br>
"7". But you could leave that for me if you want to confine this to v7A<br>
cleanup.<br></blockquote><div><br></div><div>This is a question for Tim. I agree that, to follow the rules, we should, but the AArch64 back-end might not be ready yet to generate NEON for the whole class of applications. If it is, I can easily add it.</div>
<div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2) It seems to be rather the done thing, but does getDefaultDefinitions do<br>
anything that setting VFP/NEON as subtarget features for the relevant CPUs<br>
in tabglegen wouldn't do? If not, can we kill it?<br></blockquote><div><br></div><div>This is something that crossed my mind, too. I find that function almost pointless, but I don't know what the rest of the code in the Driver does, nor how Clang uses TableGen for that kind of thing.</div>
<div><br></div><div>The issue here, I think, it's that's not for codegen or the back-end, but for Clang itself to generate the correct IR, pass on the correct flags down, etc.</div><div><br></div><div>Clang experts can chime in.</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
3) Does test/Driver/arm-mfpu.c help with some of the testing?<br></blockquote><div></div></div><br></div><div class="gmail_extra">Yes, good point. I'll complement arm-mfup and arm-cortex-cpus, should be enough.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Thanks!</div><div class="gmail_extra">--renato</div></div>