aaron.ballman added a comment. In D131012#3694502 <https://reviews.llvm.org/D131012#3694502>, @rjmccall wrote:
> I think this is fine for the ABI; the section is generally a > definition-specific property and doesn't affect use sites. > > Do we also need to check for volatile fields of records? GCC doesn't seem to do anything special there as best I can tell: https://godbolt.org/z/67eovqoG8 In D131012#3694588 <https://reviews.llvm.org/D131012#3694588>, @nickdesaulniers wrote: > Thanks for the patch. I've asked some of my colleagues who work on libabigail > their thoughts on the implications of this change. Due to timezones, it may > take some time to hear back. Mind holding this for 24hr and I'll update this > with the feedback, if any? Happy to wait! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131012/new/ https://reviews.llvm.org/D131012 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
