[PATCH] D42528: [LTO] - Introduce GlobalResolution::Prevailing flag.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 25 07:53:03 PST 2018


>I assume in D42107 you will change it back to what this patch originally did then (always setting IRName)?


Yes, Iits my plan. I'll rebase D42107 once this one be landed.


>To me it is essentially NFC either way, I don't have an issue with the original version of this patch, but will let you and Rafael >decide. (LGTM in any case, although one suggestion is to add a comment to the declaration of the isPrevailing bool field (i.e. >that IR contains the prevailing definition).

>Teresa

Will do. Thanks !

Rafael, are you ok to land latest version (with comment for isPrevailing ? suggested) ?

George.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180125/9f4c756c/attachment.html>


More information about the llvm-commits mailing list