[PATCH] D87396: Sema: add support for `__attribute__((__swift_typedef_bridged__))`
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 14 08:46:14 PDT 2020
aaron.ballman added inline comments.
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:7543
+ case ParsedAttr::AT_SwiftBridgedTypedef:
+ handleSimpleAttribute<SwiftBridgedTypedefAttr>(S, D, AL);
+ break;
----------------
compnerd wrote:
> aaron.ballman wrote:
> > compnerd wrote:
> > > aaron.ballman wrote:
> > > > Should there be any type checking that the underlying type of the typedef is a sensible one to bridge?
> > > I can't really think of anything that you could check that would be valuable. What types of things were you thinking?
> > I was mostly thinking about builtin types like `int`, but I don't insist.
> Typedefing `int` to something else and importing that with a different name is reasonable I believe. I'll add a test case.
Okay, perfect, thank you for verifying!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87396/new/
https://reviews.llvm.org/D87396
More information about the cfe-commits
mailing list