[PATCH] D75298: [Clang][SVE] Parse builtin type string for scalable vectors

Sander de Smalen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 6 10:29:01 PST 2020


sdesmalen added a subscriber: simon_tatham.
sdesmalen added a comment.
Herald added a subscriber: danielkiss.

In D75298#1904561 <https://reviews.llvm.org/D75298#1904561>, @efriedma wrote:

> > Do you happen to know which method in Sema does this? I had a look before, but couldn't find where we could do something like this.
>
> See Sema::LazilyCreateBuiltin, and its caller Sema::LookupBuiltin.


Thanks for those pointers! I spoke with @simon_tatham today, who is also very interested in this in order to simplify the MVE header file.

I would like to suggest first trying to land the current implementation (using the header file with all the declarations), so that we can use the regression tests added as part of this effort to test a different implementation that does not rely as much on the header file. What do you think?


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

https://reviews.llvm.org/D75298





More information about the cfe-commits mailing list