tra added inline comments.

================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:575
+    // times 100.
+    if (getTarget().getTargetOpts().CodeObjectVersion != "none") {
+      unsigned CodeObjVer;
----------------
When will it ever be set to `none`? Does the new option parser enforce the 
default & version values specified in the tablegen?
If so, then it should never be `none`.
If the `Values` specified for the option are not enforced, then the condition 
will be true if user specifies any invalid value other than `none`.



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

https://reviews.llvm.org/D119026

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to