[PATCH] D47145: [X86][ELF][CET] Adding the .note.gnu.property ELF section in X86

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 1 05:28:51 PDT 2018


grimar added inline comments.


================
Comment at: llvm/lib/Target/X86/X86AsmPrinter.cpp:547
+    if (M.getModuleFlag("cf-protection-branch"))
+      FeatureFlagsAnd |= ELF::GNU_PROPERTY_X86_FEATURE_1_IBT;
+    if (M.getModuleFlag("cf-protection-return"))
----------------
This `if` looks not checked by the test case?


https://reviews.llvm.org/D47145





More information about the llvm-commits mailing list