This is an archive of the discontinued LLVM Phabricator instance.

[DAGCombiner] after simplifying demanded elements of vector operand of extract, revisit the extract
ClosedPublic

Authored by spatel on Dec 13 2018, 7:26 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

spatel created this revision.Dec 13 2018, 7:26 AM
This revision is now accepted and ready to land.Dec 13 2018, 7:33 AM
This revision was automatically updated to reflect the committed changes.
spatel reopened this revision.EditedDec 13 2018, 8:38 AM

Reverted at rL349056 because:

==42943==ERROR: AddressSanitizer: use-after-poison on address 0x6210000f2a10 at pc 0x0000056013aa bp 0x7ffce91a7550 sp 0x7ffce91a7548
READ of size 8 at 0x6210000f2a10 thread T0
    #0 0x56013a9 in use_empty /b/sanitizer-x86_64-linux-fast/build/llvm/include/llvm/CodeGen/SelectionDAGNodes.h:703:35
    #1 0x56013a9 in visitEXTRACT_VECTOR_ELT /b/sanitizer-x86_64-linux-fast/build/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:15704

What is the proper way to check for node deadness here?

This revision is now accepted and ready to land.Dec 13 2018, 8:38 AM
This revision was automatically updated to reflect the committed changes.