r193515, thanks for reviewing!
================
Comment at: ../tools/clang/runtime/compiler-rt/Makefile:237
@@ -214,3 +236,3 @@
all-local:: $(RuntimeDirs:%=RuntimeLibrary.%)
-install-local:: $(RuntimeDirs:%=RuntimeLibraryInstall.%)
+install-local:: $(RuntimeDirs:%=RuntimeLibraryInstall.% RuntimeHeaderInstall)
clean-local:: CleanRuntimeLibraries
----------------
Daniel Dunbar wrote:
> Shouldn't this be:
> --
> install-local:: $(RuntimeDirs:%=RuntimeLibraryInstall.%) RuntimeHeaderInstall
> --
>
> It doesn't break, but otherwise it is expanding to RuntimeLibraryInstall.foo
> RuntimeHeaderInstall RuntimeLibraryInstall.bar RuntimeHeaderInstall
Done
================
Comment at: ../tools/clang/runtime/compiler-rt/Makefile:231
@@ +230,3 @@
+ $(PROJ_resources_include)/sanitizer
+
+RuntimeHeaderInstall: $(PROJ_resources_include)/sanitizer
----------------
Daniel Dunbar wrote:
> Might as well put these next to the PROJ_resources_lib rule earlier.
Done
================
Comment at: ../tools/clang/runtime/compiler-rt/Makefile:141
@@ -138,3 +140,3 @@
# possible.
-BuildRuntimeLibraries:
+BuildRuntimeLibraries: $(ResourceIncludeDir)/sanitizer
$(Verb) $(MAKE) -C $(COMPILERRT_SRC_ROOT) \
----------------
Daniel Dunbar wrote:
> I think it is more clear to make this a separate rule that all-local depends
> on, similar to install. It would be consistent with the rest of the header to
> have
>
> all-local:: ... RuntimeHeader
>
> RuntimeHeader: $(ResourceIncludeDir)/sanitizer
Done
http://llvm-reviews.chandlerc.com/D1959
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
