[PATCH] D91333: [clang][SveEmitter] Fix enum declarations. [NFCI]

2020-11-16 Thread Francesco Petrogalli via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG02bdbdc76021: [clang][SveEmitter] Fix enum declarations. [NFCI] (authored by fpetrogalli). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D91333: [clang][SveEmitter] Fix enum declarations. [NFCI]

2020-11-12 Thread Francesco Petrogalli via Phabricator via cfe-commits
fpetrogalli updated this revision to Diff 304941. fpetrogalli added a comment. Improve comment in test. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91333/new/ https://reviews.llvm.org/D91333 Files: clang/test/Sema/aarch64-sve-enums.c

[PATCH] D91333: [clang][SveEmitter] Fix enum declarations. [NFCI]

2020-11-12 Thread Joe Ellis via Phabricator via cfe-commits
joechrisellis added a comment. Thanks for the patch @fpetrogalli! One small comment from me, but not a blocker for me, so approved. Comment at: clang/test/Sema/aarch64-sve-enums.c:14-15 +#include +enum svpattern a1 = SV_ALL; +svpattern b1 = {1.0f}; +enum svprfop a2 =

[PATCH] D91333: [clang][SveEmitter] Fix enum declarations. [NFCI]

2020-11-12 Thread Francesco Petrogalli via Phabricator via cfe-commits
fpetrogalli created this revision. fpetrogalli added reviewers: peterwaller-arm, paulwalker-arm, joechrisellis, efriedma. Herald added subscribers: cfe-commits, kristof.beyls, tschuett. Herald added a reviewer: rengolin. Herald added a project: clang. fpetrogalli requested review of this