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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to