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

Reply via email to