steakhal added a comment.

I appreciate the effort in working on the attribute imports.

Personally, I don't really like this change. IMO 
`AttrImporter::createImportedAttr()` should do the magic, and apply the 
necessary glue for the import.
Or we could even create a dedicated function, which would 'just do the right 
thing' and import the Attr, without the current noise of calling the individual 
imports than gluing the stuff to the constructor dispatch.

The proposed implementation involves too much code repetition for me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110528

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to