[all-commits] [llvm/llvm-project] 2ee4d0: [Verifier] definition subprograms cannot be nested...
DianQK via All-commits
all-commits at lists.llvm.org
Tue Jul 25 15:08:55 PDT 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 2ee4d0386c783f58abe708298228de648239b435
https://github.com/llvm/llvm-project/commit/2ee4d0386c783f58abe708298228de648239b435
Author: DianQK <dianqk at dianqk.net>
Date: 2023-07-26 (Wed, 26 Jul 2023)
Changed paths:
M llvm/lib/IR/Verifier.cpp
A llvm/test/Verifier/disubprogram-declaration-within-struct-2.ll
A llvm/test/Verifier/disubprogram-declaration-within-struct.ll
Log Message:
-----------
[Verifier] definition subprograms cannot be nested within DICompositeType when enabling ODR.
Resolve https://github.com/llvm/llvm-project/issues/61932. We should add the validation.
LLVM can't handle IR where subprogram definitions are nested within DICompositeType when doing LTO builds, because there's no good way to cross the CU boundary to insert a nested DISubprogram definition in one CU into a type defined in another CU.
The test cases `cross-cu-inlining-2.ll` and `cross-cu-inlining-ranges.ll` can be deleted. In the `cross-cu-inlining-2.ll`, the low pc and high pc of the CU are also incorrect.
Reviewed By: dblaikie
Differential Revision: https://reviews.llvm.org/D152095
More information about the All-commits
mailing list