For record-keeping purposes, I removed this in r197729. ~Aaron
On Thu, Dec 19, 2013 at 5:09 PM, Aaron Ballman <[email protected]> wrote: > I don't mind ripping it out at all. Thank you for the information! > > ~Aaron > > On Thu, Dec 19, 2013 at 5:07 PM, John McCall <[email protected]> wrote: >> On Dec 19, 2013, at 7:13 AM, Aaron Ballman <[email protected]> wrote: >>> The ns_bridged attribute was added in r140779: >>> >>> https://llvm.org/viewvc/llvm-project?logsort=cvs&view=revision&sortby=log&revision=140779 >>> >>> This attribute creates an AST node, attaches it to a Decl, but no >>> other part of the compiler makes use of that AST node. The commit >>> message mentions that this might be interesting for tools to make use >>> of, but I am wondering whether any tools actually make use of it. From >>> what I could find on google, there doesn't appear to be much mention >>> of what this attribute is used for (if it's used at all). >>> >>> I'm trying to determine whether this attribute should continue to >>> generate an AST node, or whether this was part of an unfinished >>> feature and the attribute should be removed, or the feature finished. >>> Either is perfectly fine with me. >> >> Oops. It’s dead code. The intent was to lay the foundation for what we’re >> doing now with objc_bridge, but, well, schedules shifted, and we apparently >> forgot about having added it when we added objc_bridge later. >> >> If you wouldn’t mind ripping it out yourself, that would be great; otherwise, >> please file a bug, and somebody else will get around to it. >> >> John. _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
