tra added a comment. In D80450#2428631 <https://reviews.llvm.org/D80450#2428631>, @yaxunl wrote:
>> Also, naming. `-ffix-overload-resolution` is rather non-specific. I didn't >> mean to use it literally. The problem is that I can't think of a good >> descriptive name for what we do here. `-fgpu-fix-wrong-side-overloads` ? >> Something else? > > How about `-fgpu-exclude-wrong-side-overloads`? Since what this patch does is > always excluding wrong side overloads whereas previously only excluding wrong > side overloads if there are same side overloads. SGTM. Maybe, also make it hidden. I don't think it's useful for the end users. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80450/new/ https://reviews.llvm.org/D80450 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
