[PATCH] D24337: Fix the Thumb test for vfloat intrinsics

[Pablo Barrio via llvm-commits]

pbarrio added a comment.

On second thoughts, I have tested with a modern gcc and it marks all functions with either .arm or .thumb (equivalent to .code 16/32). So, essentially, that behaviour is what this patch suggests. Another thing that surprised me is that Thumb appears to be the default, contrary to the LLVM default which is ARM.

When you talk about preserving the "original GNU" behaviour, do you mean an old GNU toolchain? If so, why should we preserve an ancient behaviour?

Sorry if I'm missing something or this is the N-th time it is discussed on the list. If you can point me to older discussions it would be great :)


https://reviews.llvm.org/D24337