http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53664

--- Comment #12 from Ramana Radhakrishnan <ramana at gcc dot gnu.org> 
2012-07-31 17:56:29 UTC ---

(In reply to comment #8)
> For some reason I couldn't apply the patch, but manually changed the tests to
> use { scan-assembler-times regexp 2 } instead of { scan assembler regexp }.
> 
> Ramana, have you tried running the tests?  They should pass but don't.  I'll
> take a closer look at what scan-assembler-times and Tcl's regexp are doing.

I finally tried this on fsf trunk and it applied cleanly for me here and ran
the tests. The tests for vld2Q intrinsics appear to pass fine for me here. 

The changes related to vld2Q appear to be doing fine, there we've actually
removed the redundant duplication and what appears to be a bug in the test
generator. 



(In reply to comment #11)
> Sorry, I had been assuming that the tests in our tree match what's upstream 
> but
> the expressions to match are slightly different.  I'll keep investigating.


Thanks for digging further - What I don't follow is why the
scan-assembler-times appears to fail while the scan-assembler doesn't. 

FTR, the vld3Q and vld4Q intrinsics should indeed produce 2 vld3 and vld4
instructions. 

regards,
ramana

Reply via email to