Make HasWinCFI a plain bool instead of Optional<bool>
authorSanjin Sijaric <ssijaric@codeaurora.org>
Sat, 1 Sep 2018 00:33:43 +0000 (00:33 +0000)
committerSanjin Sijaric <ssijaric@codeaurora.org>
Sat, 1 Sep 2018 00:33:43 +0000 (00:33 +0000)
commit61ddb7df82361db6e30bcfa8b6ceada40a37c34b
tree5f72944529dfa8c1fcb2cf048844830d40d7bd84
parentf5297fb24bf0f21653e3fa855beb0a9e2ce81082
Make HasWinCFI a plain bool instead of Optional<bool>

Summary:
Reid suggested making HasWinCFI a plain bool defaulting to false in D50288.

It's needed in order to add HasWinCFI to MIRPrinter.  Otherwise, we'll get the
assertion:

HasWinCFI.hasValue() && "HasWinCFI not set yet!"'

Also, a few ARM64 Windows test cases will fail with the same assert if the ARM64
MCLayer part of EH work (D50166) goes in before the frame lowering part that
sets HasWinCFI (D50288 as of now).

Reviewers: rnk, mstorsjo, hans, javed.absar

Reviewed By: rnk

Subscribers: kristof.beyls, llvm-commits

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

llvm-svn: 341270
llvm/include/llvm/CodeGen/MachineFunction.h