echristo added a comment. In https://reviews.llvm.org/D35572#813520, @erichkeane wrote:
> In https://reviews.llvm.org/D35572#813369, @echristo wrote: > > > Can you update the code in CGBuiltin to use this as part of this patch? :) > > > > Thanks! > > > I'm not sure which you're referring to? I don't see a bit in CGBuiltin that > checks for a subset of the "Features", however it is a much smaller list > (since it is only things that the compiler-rt can check). Is it perhaps more > closely related to that one? It's currently checking for a subset, but that's just because it was the subset that the compiler-rt support supported. After Craig's patch I think we can just support them all unless I'm misreading something. https://reviews.llvm.org/D35572 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits