dblaikie added a comment. In D145077#4168209 <https://reviews.llvm.org/D145077#4168209>, @dblaikie wrote:
> In D145077#4162110 <https://reviews.llvm.org/D145077#4162110>, @aprantl wrote: > >> I originally was hoping we wouldn't have to introduce a new attribute for >> this, but it looks like there are legitimate concerns about the >> alternatives, so in that sense this looks good! > > Out of curiosity, what were the other existing attributes that were > considered? (would be curious to have them written down here to explain/add > strength to the motivation to add a new extension attribute) Oh, I guess this discussion might go better in the llvm side that actually introduces the attribute D145076 <https://reviews.llvm.org/D145076> Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145077/new/ https://reviews.llvm.org/D145077 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
