[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