I felt that it took a little too long getting to the point of Clara jumping in after the GI did - it was plainly obvious from the opener that that's what she was going to do, so it just felt like we were marking time waiting for her to do it. I think I'd have had more patience for the scene if the episode's opening scene hadn't been there to spell it out.
The Doctor's farewell scene with River was spot-on, though, and was the real emotional heart of the episode.
Clara had three chances to learn the Doctor's name during this episode, but it's uncertain whether she did so.
First, she got back part of her time-looped memory during their trip through the depths of the TARDIS, enough to remember the Doctor telling her about the times she'd died, but did she get it all back, including the library scene? If so, she too knows the Doctor's name, and that bit where she gets a volume of the Encyclopedia Gallifreya poured over her head may have future relevance after all.
Second, she was linked to River and may have heard River say the Doctor's name to open the tomb, if she wasn't too distracted by the GI-mook who was trying to kill her.
Third, she's seen every moment of the Doctor's time-travelling life, which may or may not include that particular personal detail - but does the Clara who the Doctor saved remember that experience?
I'm still not quite sure whether we're supposed to take from this that the Doctor's secret name is "Valeyard" or not, but I'm definitely interested to find out.