[PATCH] D130065: [X86] Use Min behavior for cf-protection-{return,branch}/ibt-seal module flags

[Xiang Zhang via Phabricator via cfe-commits]

xiangzhangllvm added inline comments.


================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:746
     // Indicate that we want to instrument return control flow protection.
-    getModule().addModuleFlag(llvm::Module::Override, "cf-protection-return",
+    getModule().addModuleFlag(llvm::Module::Min, "cf-protection-return",
                               1);
----------------
xiangzhangllvm wrote:
> Take "cf-protection-return" for example:
> I pass  the "-fcf-protection=**none**" for following test x86-cf-protection.c
> It doesn't has "!{i32 Module::Min, !"cf-protection-return", i32 **0**}".
ah! I got it at https://reviews.llvm.org/D129911 line 1489, thanks


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130065/new/

https://reviews.llvm.org/D130065