rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land.
lgtm, thanks! ================ Comment at: clang/lib/CodeGen/CGCall.cpp:1973 } - if (!AttrOnCallSite && TargetDecl->hasAttr<NoMergeAttr>()) FuncAttrs.addAttribute(llvm::Attribute::NoMerge); ---------------- This is the key change, I think this is worth a comment. Now we only place LLVM nomerge attributes on call sites, never functions, and this allows them to work on indirect virtual function calls. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94537/new/ https://reviews.llvm.org/D94537 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits