HomePhabricator

Merging r339822:

Description

Merging r339822:

r339822 | carrot | 2018-08-16 00:08:26 +0200 (Thu, 16 Aug 2018) | 12 lines

[CodeGenPrepare] Add BothExtension type to PromotedInsts

This patch fixes PR38125.

Instruction extension types are recorded in PromotedInsts, it can be used later in function canGetThrough. If an instruction has two users with different extension types, it will be inserted into PromotedInsts two times in function promoteOperandForOther. The second one overwrites the first one, and the final extension type is wrong, later causes problem in canGetThrough.

This patch changes the simple bool extension type to 2-bit enum type, add a BothExtension type in addition to zero/sign extension. When an user sees BothExtension for an instruction, it actually knows nothing about how that instruction is extended.

Differential Revision: https://reviews.llvm.org/D49512


Details

Committed
hansAug 21 2018, 9:15 AM
Differential Revision
D49512: [CodeGenPrepare] Add BothExtension type to PromotedInsts
Parents
rL340293: lldbtest.py: Unconditionally set the clang module cache path.
Branches
Unknown
Tags
Unknown