timshen added a comment.

In https://reviews.llvm.org/D27872#639020, @hfinkel wrote:

> it is not at all obvious what is going on (i.e. why are we casting to 
> integers?). Maybe semPPCDoubleDoubleImpl needs a better name now? It seems 
> confusing to have semPPCDoubleDoubleImpl and semPPCDoubleDouble where one is 
> not really an "implementation" class of the other.


This is true. I renamed it to semPPCDoubleDoubleLegacy, and modified its 
documentation. I didn't put the documentation for each of the uses, because 
that'd be too verbose.


https://reviews.llvm.org/D27872



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to