Skip to content

Commit 24e249a

Browse files
chinmaygardednfield
authored andcommitted
Fix top-level build group to not include the compiler and fixup headers. (flutter#66)
Targets in the engine had to individually depend on each sub-target in Impeller when all they cared about was all the client libs. The compiler is a build time implicit dependency. So that has been removed from the top-level group. Also fixed all the headers so that TUs within Impeller don't care about where Impeller itself sits in the source tree.
1 parent e5fc548 commit 24e249a

File tree

16 files changed

+40
-49
lines changed

16 files changed

+40
-49
lines changed

impeller/BUILD.gn

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,9 @@
33
# found in the LICENSE file.
44

55
config("impeller_public_config") {
6-
include_dirs = [
7-
"..",
8-
".",
9-
]
6+
include_dirs = [ ".." ]
107
}
118

12-
is_host = is_mac || is_linux || is_win
13-
149
group("impeller") {
1510
public_deps = [
1611
"aiks",
@@ -24,10 +19,6 @@ group("impeller") {
2419
"tessellator",
2520
"typographer",
2621
]
27-
28-
if (is_host) {
29-
public_deps += [ "compiler" ]
30-
}
3122
}
3223

3324
executable("impeller_unittests") {

impeller/aiks/image.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
#include "flutter/impeller/aiks/image.h"
5+
#include "impeller/aiks/image.h"
66

77
namespace impeller {
88

impeller/aiks/picture_recorder.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
#include "flutter/impeller/aiks/picture_recorder.h"
5+
#include "impeller/aiks/picture_recorder.h"
66

7-
#include "flutter/impeller/aiks/canvas.h"
7+
#include "impeller/aiks/canvas.h"
88

99
namespace impeller {
1010

impeller/base/validation.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
#include "flutter/impeller/base/validation.h"
5+
#include "impeller/base/validation.h"
66

77
#include "flutter/fml/logging.h"
88

impeller/compiler/compiler.cc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,14 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
#include "flutter/impeller/compiler/compiler.h"
5+
#include "impeller/compiler/compiler.h"
66

77
#include <filesystem>
88
#include <memory>
99
#include <sstream>
1010

1111
#include "flutter/fml/paths.h"
12-
#include "flutter/impeller/compiler/logger.h"
12+
#include "impeller/compiler/logger.h"
1313

1414
namespace impeller {
1515
namespace compiler {

impeller/compiler/compiler.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010

1111
#include "flutter/fml/macros.h"
1212
#include "flutter/fml/mapping.h"
13-
#include "flutter/impeller/compiler/include_dir.h"
14-
#include "flutter/impeller/compiler/reflector.h"
13+
#include "impeller/compiler/include_dir.h"
14+
#include "impeller/compiler/reflector.h"
1515
#include "shaderc/shaderc.hpp"
1616
#include "third_party/spirv_cross/spirv_msl.hpp"
1717
#include "third_party/spirv_cross/spirv_parser.hpp"

impeller/compiler/compiler_unittests.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,10 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
#include "flutter/impeller/compiler/compiler.h"
65
#include "flutter/testing/testing.h"
76
#include "gtest/gtest.h"
87
#include "impeller/base/validation.h"
8+
#include "impeller/compiler/compiler.h"
99

1010
namespace impeller {
1111
namespace compiler {

impeller/compiler/impellerc_main.cc

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,9 @@
88
#include "flutter/fml/file.h"
99
#include "flutter/fml/macros.h"
1010
#include "flutter/fml/mapping.h"
11-
#include "flutter/impeller/compiler/compiler.h"
12-
#include "flutter/impeller/compiler/switches.h"
13-
#include "flutter/impeller/compiler/utilities.h"
11+
#include "impeller/compiler/compiler.h"
12+
#include "impeller/compiler/switches.h"
13+
#include "impeller/compiler/utilities.h"
1414
#include "third_party/shaderc/libshaderc/include/shaderc/shaderc.hpp"
1515

1616
namespace impeller {

impeller/compiler/reflector.cc

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
#include "flutter/impeller/compiler/reflector.h"
5+
#include "impeller/compiler/reflector.h"
66

77
#include <atomic>
88
#include <optional>
@@ -11,12 +11,12 @@
1111

1212
#include "flutter/fml/closure.h"
1313
#include "flutter/fml/logging.h"
14-
#include "flutter/impeller/compiler/code_gen_template.h"
15-
#include "flutter/impeller/compiler/utilities.h"
16-
#include "flutter/impeller/geometry/matrix.h"
17-
#include "flutter/impeller/geometry/scalar.h"
1814
#include "impeller/base/strings.h"
1915
#include "impeller/base/validation.h"
16+
#include "impeller/compiler/code_gen_template.h"
17+
#include "impeller/compiler/utilities.h"
18+
#include "impeller/geometry/matrix.h"
19+
#include "impeller/geometry/scalar.h"
2020

2121
namespace impeller {
2222
namespace compiler {

impeller/compiler/switches.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
#include "flutter/impeller/compiler/switches.h"
5+
#include "impeller/compiler/switches.h"
66

77
#include <filesystem>
88
#include <map>

0 commit comments

Comments
 (0)