[LowerTypeTests] Avoid creation of select constant expression
authorNikita Popov <npopov@redhat.com>
Fri, 3 Mar 2023 15:22:41 +0000 (16:22 +0100)
committerNikita Popov <npopov@redhat.com>
Mon, 6 Mar 2023 08:49:40 +0000 (09:49 +0100)
commit0317147a2848547ec97d8e76782f7dc38267a21f
tree83ef8e75f76b0cf6e15bf4ccc48237eed2c151f1
parent0354463b9e77b909ec54526aa0375750b77c4282
[LowerTypeTests] Avoid creation of select constant expression

LowerTypeTests replaces weak declarations with an icmp+select
constant expressions. As this is not a relocatable expression,
it additionally promotes initializers using it to global ctors.

As part of https://discourse.llvm.org/t/rfc-remove-most-constant-expressions/63179,
I would like to remove the select constant expression, of which LTT
is now the last user. This is a bit tricky, because we now need to
replace a constant with an instruction, which might require
converting intermediate constant expression users to instructions as
well.

We do this using the convertUsersOfConstantsToInstructions() helper.
However, it needs to be slightly extended to also support expansion
of ConstantAggregates. These are important in this context, because
the promotion of initializers to global ctors will produce stores
of such aggregates.

Differential Revision: https://reviews.llvm.org/D145247
llvm/lib/IR/ReplaceConstant.cpp
llvm/lib/Transforms/IPO/LowerTypeTests.cpp
llvm/test/Transforms/LowerTypeTests/cfi-direct-call1.ll
llvm/test/Transforms/LowerTypeTests/function-weak.ll