[PATCH] D76679: [SveEmitter] Add more immediate operand checks.

Sander de Smalen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 3 09:08:06 PDT 2020


sdesmalen added a comment.

In D76679#1957399 <https://reviews.llvm.org/D76679#1957399>, @SjoerdMeijer wrote:

> I think the float16 discussion is an interesting one, but doesn't necessarily need to be done here. I am asking some questions offline, but if we ever come to a different opinion on it, then we can follow up so it's somewhat orthogonal to this change, and so this looks fine to me.


Thanks for reviewing the patch Sjoerd!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76679





More information about the cfe-commits mailing list