r197224 - Enabling thumb2 mode used to force support for armv6t2. Replace this

Bernie Ogden bogden at arm.com
Mon Dec 16 08:15:47 PST 2013


Hi Joerg,

Just looking at this change -
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20131209/198615.
html (I can't reply directly as the original mail didn't reach my account).

Perhaps I'm just failing to understand, but I don't understand why you need
to add an explicit CPU to all of these tests. If you're testing that CPUs
with Thumb2 also have V6T2Ops then wouldn't it be better to add a single
test that explicitly checks for this?

Regards,

Bernie







More information about the llvm-commits mailing list