[PATCH] D113107: Support of expression granularity for _Float16.

Zahira Ammarguellat via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 23 11:41:06 PDT 2022


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



More information about the llvm-commits mailing list