hfinkel added a comment. In https://reviews.llvm.org/D24909#559110, @SjoerdMeijer wrote:
> Hi Hal, > Thanks for reviewing and you're right: this should work. We actually have > actually some downstream (aarch64) build attribute selection code that would > work better with this change. Are you okay with committing this change? > Cheers, > Sjoerd. Can you explain what you mean? What does this code do? My general impression is that we're planning to rip out this code entirely and reply only on setting the functions attributes. @echristo, am I right about this? https://reviews.llvm.org/D24909 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
