[PATCH] D31286: [X86] Fix Stale SDNode use in X86ISelDAGtoDAG

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

The fix seems trivial and looks good. Please submit to at least fix the crasher.

One note for this and future, similar patches: can you minimize the test case any? At the least stripping off some of the metadata cruft and comment boiler plate would be good.

Also, please always run FileCheck and assert that reasonable instructions are generated. "doesn't crash" isn't the most useful test case, and small cases like this should be easy to check that sane instructions come out.


https://reviews.llvm.org/D31286