[PATCH] D81911: [ThinLTO] Work around getBaseObject returning null for alias-to-ifunc

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 8 08:03:55 PDT 2020


tejohnson added a comment.

In D81911#2259461 <https://reviews.llvm.org/D81911#2259461>, @nextsilicon-itay-bookstein wrote:

> I have updated the diff to include the changes you suggested. However, as you said, for the test to have a chance of passing, D82745 <https://reviews.llvm.org/D82745> is required. I will send out a mail to discuss the design issue on llvm-dev soon.

If the code won't actually work for now, does it even make sense to submit this one before D82745 <https://reviews.llvm.org/D82745> and the general issue are resolved? Perhaps this one should be made dependent on D82745 <https://reviews.llvm.org/D82745>.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81911/new/

https://reviews.llvm.org/D81911



More information about the llvm-commits mailing list