[PATCH] [ARM] ARM attribute parser incorrectly handles CPU_arch_profile

Charlie Turner charlie.turner at arm.com
Mon Nov 24 13:32:48 PST 2014


Hi Tim, thanks for taking a look!

>>! In D6341#5, @t.p.northover wrote:
> Shouldn't this be testable?

It is, as I mention in the test plan, the large suite of tests I have in D6319 will cover this attribute for all permissible values. It is those tests that uncovered this bug. I would like to fix this first to avoid an erroneous test failure when D6319 is committed.

Kind regards,
Charlie.

http://reviews.llvm.org/D6341






More information about the llvm-commits mailing list