<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 Nov 14, 2013, at 8:59 AM, Bob Wilson <<a href="mailto:bob.wilson@apple.com">bob.wilson@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="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;"><br>On Nov 14, 2013, at 2:55 AM, Bradley Smith <<a href="mailto:bradley.smith@arm.com">bradley.smith@arm.com</a>> wrote:<br><br><blockquote type="cite"><blockquote type="cite"><blockquote type="cite">Ok, if your main concern is to get rid of the cortex-a9-mp cpu, then<br></blockquote>I would follow Renato's suggestion of just adding a -mattr=+mp option.<br>I don't see a bigger change like you suggested below is worth it.<br><blockquote type="cite"><br>Evan<br></blockquote><br>That would work, but it would be good to clean up this stuff a bit and<br>I think Bradley is on the right track.<br><br>First off, the issue here is specifically v7f. It has nothing to do<br>with either v7s or v7k. I will need to double check, but I'm not aware<br>of any fundamental reason why we need to keep v7f around. If we remove<br>v7f, then I'm not aware of any reason to keep the cortex-a9-mp cpu.<br></blockquote><br>Our immediate concern is indeed just v7f, consistency says that whatever we<br>do to v7f we do to the others, however that depends entirely on others<br>requirements, we don't have a requirement to do anything to v7s and v7k so<br>can ignore those if needed.<br><br>For v7f, are you suggesting that we take the approach of adding a CPUSubType<br>option to pass information through to LLVM? Or that we may be able to get<br>rid of it entirely and forget about CPUSubType generation for v7f<br>specifically?<br></blockquote><br>I am suggesting that we completely remove the armv7f architecture along with the cortex-a9-mp cpu. I need to do some checking first to make sure no one is using that. It will take a little while to do that checking, but I think that would be the right solution. I will get back to you.</div></blockquote></div><br><div>As expected, it took a while, but I have now removed armv7f. I kept the code to recognize the cortex-a9-mp cpu name, but changed to match to the base armv7 architecture.</div><div><br></div><div>clang svn r199336</div><div>llvm svn r199337</div><div>compiler-rt svn r199333</div><div><br></div><div>Hopefully that will unblock the other changes that you wanted to make.</div></body></html>