[llvm] r210920 - Finishing touch for the std::error_code transition.
Jeffrey Yasskin
jyasskin at google.com
Fri Jun 13 13:40:59 PDT 2014
On Fri, Jun 13, 2014 at 7:45 PM, Chandler Carruth <chandlerc at google.com> wrote:
>
> On Fri, Jun 13, 2014 at 6:42 PM, Hal Finkel <hfinkel at anl.gov> wrote:
>>
>> > +// * std::errc is a namespace in some implementations. That meas
I'm not sure exactly what Rafael means here, but it sounds like those
implementations don't match the standard. [syserr] requires std::errc
to be an enum class, not a namespace.
>> > that ADL
>> > +// doesn't work and it is sometimes necessary to write
>> > std::make_error_code
>> > +// or in templates:
>> > +// using std::make_error_code;
>> > +// make_error_code(...);
>> > +//
>> > +// with this enum it is safe to always just use make_error_code.
>>
>> Should we file a defect report?
>
>
> Marshall or Jeffrey would know better.
Defect reports about ADL working badly are either a lost cause, or a
commitment to a huge amount of work.
More information about the llvm-commits
mailing list