sdesmalen added inline comments.
================ Comment at: clang/lib/CodeGen/CGBuiltin.cpp:7451 + } + llvm_unreachable("Unknown MemEltType"); +} ---------------- SjoerdMeijer wrote: > nit: to be consistent, do this in the default clause? Doing that would lead to warning: default label in switch which covers all enumeration values [-Wcovered-switch-default] Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77735/new/ https://reviews.llvm.org/D77735 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits