aaron.ballman added inline comments.
================
Comment at: test/clang-tidy/misc-suspicious-enum-usage.cpp:122
+struct a<f<ad, ae, af>> {
+ enum { ah = ad::m,
+ ai = ae::m,
----------------
alexfh wrote:
> alexfh wrote:
> > aaron.ballman wrote:
> > > alexfh wrote:
> > > > aaron.ballman wrote:
> > > > > This seems like a lot of complicated code for the test case -- can
> > > > > this be reduced further?
> > > > This is the best creduce could do. It should be possible to make this
> > > > much shorter, but I wouldn't spend too much time on that.
> > > I'd appreciate the time spent because this is an almost-unintelligible
> > > test for anyone reading it -- it's hard to understand what's going on
> > > there.
> > namespace PR34400 {
> > enum { E1 = 0 };
> > enum { E2 = -1 };
> >
> > enum { l = E1 | E2 };
> > }
> >
> This might not cover both code paths affected by the bug. But it should be
> easy to construct a case for the second one.
It's certainly an easier case to grok, thank you! :-D
https://reviews.llvm.org/D37572
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
