Identified in D109359.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
@jansvoboda11 this came up in D109359 with warning:
llvm/include/llvm/Option/OptParser.td:217:91: warning: unused template argument: MarshallingInfoBooleanFlag:name
I see you've changed worked on this in the past, wasn't sure if it's intended for name not to be used so thought I'd post the patch and let you take a look.
Comment Actions
LGTM.
To give more background, I think my intention was to add an assertion that name and other_name are the same (except for the no_ part) and I was waiting for TableGen assertions to land. But I don't think that's too valuable, since the callers already ensure that by some other mechanism. This should be fine to remove.