Not bad, he looks enough like him to get the idea across and he is a fun actor who could have given the role the necessary coldness.
Nobody seems to care that Ewan doesn't look or even sound exactly like a younger Sir Alec.
If they were serious about putting Tarkin in the prequels they should of written a role for him and hired someone like Nighy who could have played him without OTT make-up.