diff options
| author | Craig Topper <craig.topper@intel.com> | 2019-04-08 19:44:07 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2019-04-08 19:44:07 +0000 |
| commit | 3a4c2192a471cdc90dda914d734a0cb837a4d462 (patch) | |
| tree | 4b5b103a8abacd43993d5fffad822ded31495ead /llvm/lib/ExecutionEngine/Orc | |
| parent | 6ed5706a2beab0bcde2a24e36751ea30cd5b4009 (diff) | |
| download | bcm5719-llvm-3a4c2192a471cdc90dda914d734a0cb837a4d462.tar.gz bcm5719-llvm-3a4c2192a471cdc90dda914d734a0cb837a4d462.zip | |
[X86] Fix a couple lowering functions that called ReplaceAllUsesOfValueWith for the newly created code and then return SDValue(). Use MERGE_VALUES instead.
Returning SDValue() makes the caller think custom lowering was unsuccessful and then it will fall back to trying to expand the original node. This expanded code will end up with no users and end up being pruned later. But it was useless unnecessary work to create it.
Instead return a MERGE_VALUES with all the results so the caller knows something changed. The caller can handle the replacements.
For one of the cases I had to use UNDEF has a dummy value for a result we know is unused. This should get pruned later.
llvm-svn: 357935
Diffstat (limited to 'llvm/lib/ExecutionEngine/Orc')
0 files changed, 0 insertions, 0 deletions
