HazardyKnusperkeks accepted this revision. HazardyKnusperkeks added a comment. This revision is now accepted and ready to land.
In D149562#4337955 <https://reviews.llvm.org/D149562#4337955>, @sstwcw wrote: >> I'll go along with other reviewers on this one. > > So what do the other reviewers think about this patch? The braced > initialization list thing was added a long time ago by DJasper, so that > behavior probably has to stay. I'm not particularly fond of that feature, but you are right, it has to stay. So making it consistent in verilog doesn't seem to be a problem to me. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149562/new/ https://reviews.llvm.org/D149562 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits