aaron.ballman added a comment.

This looks correct per my reading of [basic.start.dynamic], but is this an ABI 
breaking change that we may want to use ABI versioning for in case someone is 
relying on the old order for some reason?

Also, the change should have a release note for the fix.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127233

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

Reply via email to