tamas.petz added inline comments.
================ Comment at: clang/lib/CodeGen/CGCall.cpp:1827-1828 + + auto RASignKind = getLangOpts().getSignReturnAddressScope(); + if (RASignKind != LangOptions::SignReturnAddressScopeKind::None) { + FuncAttrs.addAttribute("sign-return-address", ---------------- LangOptions has the following functions: - hasSignReturnAddress() - isSignReturnAddressWithAKey() - isSignReturnAddressScopeAll() With these functions some of these lines can be significantly reduced. ================ Comment at: clang/lib/CodeGen/CGCall.cpp:1837 + "sign-return-address-key", + RASignKey == LangOptions::SignReturnAddressKeyKind::AKey ? "a_key" + : "b_key"); ---------------- Use getLangOpts(). isSignReturnAddressWithAKey() ================ Comment at: clang/lib/CodeGen/CodeGenModule.cpp:596 + + LangOptions::SignReturnAddressScopeKind Scope = + Context.getLangOpts().getSignReturnAddressScope(); ---------------- Use new functions please. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75181/new/ https://reviews.llvm.org/D75181 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits