[mlir][llvm] Fix bug in the LLVM IR constant import.
authorTobias Gysi <tobias.gysi@nextsilicon.com>
Wed, 14 Dec 2022 09:27:01 +0000 (10:27 +0100)
committerTobias Gysi <tobias.gysi@nextsilicon.com>
Wed, 14 Dec 2022 09:27:12 +0000 (10:27 +0100)
commit3674cadf4d6a3390875bd0173470f3a55870edcd
tree8b4298def3740d676ed809395ead30bc8c529ad7
parent1da4d5aafad74a3d9a3b893335249c9e5b4a7ab9
[mlir][llvm] Fix bug in the LLVM IR constant import.

The recently introduced iterative constant import
(https://reviews.llvm.org/D137559) fails for programs that
subsequently import constant expressions with duplicate
subexpressions. The reason is a broken duplicate check
in getConstantsToConvert. The revision fixes the bug and
adds a test case that imports two constant expressions
with duplicates.

Reviewed By: ftynse

Differential Revision: https://reviews.llvm.org/D139918
mlir/lib/Target/LLVMIR/ConvertFromLLVMIR.cpp
mlir/test/Target/LLVMIR/Import/constant.ll