[PATCH] D60573: [Sema] ADL: Associated namespaces for class types and enumeration types (CWG 1691)

Thu Apr 11 17:21:32 PDT 2019

riccibruno added a comment.

In D60573#1463611 <https://reviews.llvm.org/D60573#1463611>, @rsmith wrote:

> > I have applied the fix to all language versions, even though I think that the DR only applies to C++14
>
> DRs don't have a specific version that they are intended to apply to; that's up to us to determine, and generally we apply them retroactively as far back as they make sense -- they are supposed to be bug fixes, after all. (>From ISO's perspective, C++11 was retracted and replaced by C++14 when the latter was published, so it's simply not meaningful for a DR to officially target anything older than that, and the only version that DRs can meaningfully list is whatever version was most recently published when they were approved.)
>
> This looks good to me. Thanks!

Good to know, thanks for the explanation and review !

Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60573/new/

https://reviews.llvm.org/D60573