rnk added a comment.

In D91659#2458639 <https://reviews.llvm.org/D91659#2458639>, @rsmith wrote:

> @rnk Your thoughts on this would be appreciated.

I think it's important to make MIDL happy if we can. As to how to restructure 
the code to get there, I took a look, but wasn't able to come up with any 
helpful suggestions without putting a lot more time into this.

What is the issue with the current structure, ActOnTag produces the wrong AST 
node, an ElaboratedType, but we want to produce a TypedefType?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91659

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D91659: A... Shivanshu Goyal via Phabricator via cfe-commits
    • [PATCH] D916... Shivanshu Goyal via Phabricator via cfe-commits
    • [PATCH] D916... Shivanshu Goyal via Phabricator via cfe-commits
    • [PATCH] D916... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D916... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D916... Reid Kleckner via Phabricator via cfe-commits
    • [PATCH] D916... Richard Smith - zygoloid via Phabricator via cfe-commits

Reply via email to