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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits