[PATCH] D157826: [X86] Allow inlining callees missing VLX feature

Kal Conley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 13 15:00:22 PDT 2023

kalcutter added inline comments.

Comment at: llvm/lib/Target/X86/X86TargetTransformInfo.cpp:6082
         // Do a precise compatibility check.
         if (!areTypesABICompatible(Caller, NestedCallee, Types))
           return false;
nikic wrote:
> I assume that this is the actually failing check? In that case, should the adjustment be in that function?
I am not sure. That function is checking that the used types are ABI compatible, but what about available target instructions? Is the idea to first only check the types, then later during CodeGen the target does a more exhaustive check?

Fixing areTypesABICompatible() would be more general I guess. Do you have an idea how much work is involved properly fixing that function? I am not familiar with this code base at all. Do you have anything against applying this patch as an incremental improvement?



More information about the llvm-commits mailing list