aaron.ballman added inline comments.
================
Comment at: clang/test/AST/ast-dump-enum-bool.cpp:1
+// RUN: %clang_cc1 -triple x86_64-unknown-unknown -fsyntax-only -ast-dump=json 
%s | FileCheck %s
+
----------------
riccibruno wrote:
> Why a (pretty unreadable) json test? There is also `make-ast-dump-check.sh` 
> to auto-generate an ast dump test.
I'd also like to see some CodeGen tests for various fixed underlying types that 
ensures the conversion to the underlying type happens before the conversion to 
the enumeration.

Also, at least one test showing this behaves properly with a C-style cast 
instead of a named cast.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85612

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

Reply via email to