nickdesaulniers added inline comments.
================
Comment at: clang/include/clang/Basic/LangOptions.h:60
enum GCMode { NonGC, GCOnly, HybridGC };
- enum StackProtectorMode { SSPOff, SSPOn, SSPStrong, SSPReq };
+ enum StackProtectorMode { SSPOff, SSPOn, SSPStrong, SSPReq, SSPUnspecified };
----------------
void wrote:
> I don't know if it's a convention, but an unspecified enum is typically put
> at the beginning of the enum list.
This ordering produces the least amount of churn. If I change the ordering,
then all of the enum values need to be incremented across all unit tests that
specify `-stack-protector <N>` to `cc1`. It would simplify the use of
`std::max` in `RenderSSPOptions` slightly. WDYT?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90194/new/
https://reviews.llvm.org/D90194
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
