[PATCH] D52550: [ARM] Check for sel intrinsic use in ARM CGP

[Sam Parker via Phabricator via llvm-commits]

samparker added a comment.

Hi Eli,

I don't intend to remove the -arm-enable-scalar-dsp flag or change the default from false, meaning that the user will have to enable it and even then the pass can decide against it. From your comments in https://reviews.llvm.org/D48832 and John's in  https://reviews.llvm.org/D49239, this seems like the most conservative approach to me. By the ACLE standard I don't have to worry about the whole program and I could just look at the current function. I will want to use the same approach in the parallel dsp pass but I want to be as safe as possible.


https://reviews.llvm.org/D52550