<div dir="ltr">+cfe-dev, I must have dropped it<br><div class="gmail_extra"><br></div><div class="gmail_extra">Richard says we shouldn't build AttributedTypes for this, even though the comments there suggest it should be used for that. The AttributedType modified type is redundant with the VectorType element type.</div><div class="gmail_extra"><br></div><div class="gmail_extra">So, I think beefing up the implementation of VectorTypeLoc is the way to go, which seems like what you were already doing.</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 21, 2016 at 3:42 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I don't think you want to go down the path of adding VisitVectorTypeLoc. Users can't "write" vector types directly, they can only write an AttributedType which desugars to a vector type and modifies the underlying type written in the source code. Type locations are supposed to model types that the user actually wrote.<div><br></div><div>You want to build a TypeLoc that's similar in structure to what you'd get for 'int (*vs_test);', which I think is a PointerTypeLoc around a ParenTypeLoc around an BuiltinTypeLoc.</div><div><br></div><div>I'm still looking at this, but that's what I think I know about this area. </div></div></blockquote></div></div></div>