[x86][slh][NFC] Rm redundant liveness check
authorGeorge Burgess IV <george.burgess.iv@gmail.com>
Tue, 10 Mar 2020 00:02:51 +0000 (17:02 -0700)
committerGeorge Burgess IV <george.burgess.iv@gmail.com>
Tue, 10 Mar 2020 00:07:44 +0000 (17:07 -0700)
commitbb0ec1daff1b0eee4936878749365e625e71182c
treed499ab0f09a0289ddf46b86e298eebc785a0763e
parent6333cc2a12dc72d0cfaaac5ade00e9a1b9d6dbce
[x86][slh][NFC] Rm redundant liveness check

Patch by Zola Bridges!

From the review:

"""
In this changeset (https://reviews.llvm.org/D70283), I added a liveness
check everywhere the isDataInvariant* functions were used, so that I
could safely delete the checks within the function. I mistakenly left
that deletion out of the patch. The result is that the same condition is
checked twice for some instructions which is functionally fine, but not
good. This change deletes the redundant check that I intended to delete
in the last change.

This is the second of three patches that will make the data invariance
checks available for non-SLH passes and enable the FIXMEs related to
moving this metadata to the instruction tables to be resolved.

Tested via llvm-lit llvm/test/CodeGen/X86/speculative-load-hardening*
"""

Differential Revision: https://reviews.llvm.org/D75650
llvm/lib/Target/X86/X86SpeculativeLoadHardening.cpp