On Fri, Jan 27, 2017 at 2:11 PM Mehdi AMINI via Phabricator < revi...@reviews.llvm.org> wrote:
> mehdi_amini accepted this revision. > mehdi_amini added a comment. > This revision is now accepted and ready to land. > > LGTM. > > > > ================ > Comment at: lib/AST/ASTContext.cpp:8909 > + > + // Itanium ABI (& MSVC seems to do similarly) requires static locals > in > + // inline functions to be emitted anywhere they're needed, even if the > ---------------- > I assume you looked it up, do you have a ref? (Citation or pointer to > right section/paragraph). > Would you like a citation in source? I thought that might be a bit strong since this is ABI-neutral code, notionally (so I was straddling that line a bit). I can't quite figure out how to navigate/find the ABI document nor cast the runes as Richard did for the citation, so hopefully he can chime in here. > > > https://reviews.llvm.org/D29233 > > > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
