[PATCH] D71486: [clang-tools-extra] Fix switch coverage warning

Dávid Bolvanský via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 13 13:18:39 PST 2019


xbolva00 added a comment.

In D71486#1784079 <https://reviews.llvm.org/D71486#1784079>, @sammccall wrote:

> It should be possible to test this by adapting a test case from the original ed8dadb <https://reviews.llvm.org/rGed8dadb37c7e1a7f4889d868ac9b19bfe7762237>, but I'm not certain...


This concern was not addressed/answered.

The commit https://reviews.llvm.org/rG5623bd52acd34db2e9cfc11d1510407610a14db0 claims that this was reviewed by rsmith, but I don't see rsmith's LGTM here..


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71486





More information about the cfe-commits mailing list