[PATCH] D40478: Added control flow architecture protection Flag

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: lib/CodeGen/CodeGenModule.cpp:501
+  if (CodeGenOpts.CFProtectionReturn) {
+    Target.checkCFProtectionReturnSupported(getDiags());
+    // Indicate that we want to instrument return control flow protection.
----------------
Should we still be adding the module flag if the target says its not supported?


Repository:
  rL LLVM

https://reviews.llvm.org/D40478