zahiraam added a comment.

In D113107#3743318 <https://reviews.llvm.org/D113107#3743318>, @rjmccall wrote:

> Somehow we've taken a huge step back on unpromotion, and I'm worried you're 
> now doing the exact thing I didn't want us doing and forcing all the 
> downstream clients to handle the possibility of a promoted result.

Okay may be trying to do the optimization that @pengfei  was mentioning is too 
soon to do? Let me backup here and let for the Unary operators do the 
promote/unpromote back to back and see if that's better.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113107/new/

https://reviews.llvm.org/D113107

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

Reply via email to