ahatanak added inline comments.

================
Comment at: lib/Sema/SemaStmt.cpp:1165
 
       if (!hasCasesNotInSwitch)
         SS->setAllEnumCasesCovered();
----------------
This function used to call setAllEnumCasesCovered() when parsing a switch 
statement with an opaque enum condition, but this patch changes the behavior. I 
couldn't tell whether isAllEnumCasesCovered should return true or false for 
such switch statements, but was this change intentional?


Repository:
  rL LLVM

https://reviews.llvm.org/D27299



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

Reply via email to