Skip to content

Commit e175983

Browse files
committed
Add a couple missing Optional includes
To avoid synced PRs, add some temporary `Optional.h` includes so that the move from `llvm::Optional` -> `std::optional` in LLVM can be merged. Once the LLVM change is in we can merge the Swift move. Then we'll need one final PR to fully remove `Optional.h` and `None.h` (this will likely have to be a synced PR since optional references keep being added).
1 parent 2875980 commit e175983

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

include/swift/AST/IRGenOptions.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
#include "swift/Basic/OptimizationMode.h"
2626
#include "swift/Config.h"
2727
#include "clang/Basic/PointerAuthOptions.h"
28+
#include "llvm/ADT/Optional.h"
2829
#include "llvm/IR/CallingConv.h"
2930
// FIXME: This include is just for llvm::SanitizerCoverageOptions. We should
3031
// split the header upstream so we don't include so much.

include/swift/Driver/Driver.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
#include "swift/Basic/Sanitizers.h"
2626
#include "swift/Driver/Util.h"
2727
#include "llvm/ADT/DenseMap.h"
28+
#include "llvm/ADT/Optional.h"
2829
#include "llvm/ADT/StringRef.h"
2930

3031
#include <functional>

0 commit comments

Comments
 (0)