ldionne added a comment.

In D31413#2279182 <https://reviews.llvm.org/D31413#2279182>, @aaron.ballman 
wrote:

> Yes, and the way I would handle this is to change the `init_priority` value 
> checking to allow values <= 100 if the attribute location is within a system 
> header. This would allow libc++ to use the reserved priority but would still 
> give errors when attempted from user code.

Ok, that makes sense. If Clang implements that change, I can go back and change 
it in libc++. I unfortunately don't have bandwidth to go and do that, since 
this isn't affecting a platform I directly work on. So if anyone's willing to 
go and change Clang to make that work, ping me here and I'll fix libc++.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D31413

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

Reply via email to