This is to comply with LLVM project policy. For any license header that mentioned a specific copyright owner, the owner has been added to to CREDITS.txt.
Diff Detail
Event Timeline
Looks reasonable, please add the standard LLVM file header blob per the coding standards.
See my comment for gen_convert.py. I think we're accidentally deleting more than we mean to.
generic/lib/gen_convert.py | ||
---|---|---|
28 | Did you mean to remove this part of the script's header comment? |
I've gotten to the point of hitting a build error due to the addition/removal of BitcodeReader.h at some point since the last time I built llvm on this machine.
In general
- we need to make sure that the she-bangs in python scripts don't get moved down from the first line in the file.
- The comment format in python files isn't //, it's #
- The llvm assembly (*.ll) comment format is also wrong
- The SOURCES files are treated as literal lists of files, and the copyright statement in them is breaking metabuild.py. Either remove the copyright header from those files, or fix metabuild to ignore lines starting with a chosen comment character(s).
amdgcn-amdhsa/lib/SOURCES | ||
---|---|---|
1 ↗ | (On Diff #86411) | build/metabuild.py errors out here... It treats all lines in the SOURCES files as entries and tries to make directories for anything it thinks needs a parent directory... |
amdgcn/lib/SOURCES | ||
1 ↗ | (On Diff #86411) | build/metabuild.py errors out here... It treats all lines in the SOURCES files as entries and tries to make directories for anything it thinks needs a parent directory... |
amdgpu/lib/SOURCES | ||
1 ↗ | (On Diff #86411) | build/metabuild.py errors out here... It treats all lines in the SOURCES files as entries and tries to make directories for anything it thinks needs a parent directory... |
build/metabuild.py | ||
2 | This isn't valid python syntax anymore. Prefix the license lines with '#'. | |
build/ninja_syntax.py | ||
2 | This isn't valid python syntax anymore. Prefix the license lines with '#'. Also, you've moved the shebang down from the first line in the file, which means that this ends up getting treated as a shell script instead of python. | |
configure.py | ||
2 | This isn't valid python syntax anymore. Prefix the license lines with '#'. Also, you've moved the shebang down from the first line in the file, which means that this ends up getting treated as a shell script instead of python. | |
generic/lib/SOURCES | ||
1 ↗ | (On Diff #86411) | build/metabuild.py errors out here... It treats all lines in the SOURCES files as entries and tries to make directories for anything it thinks needs a parent directory... |
generic/lib/atomic/atomic_impl.ll | ||
2 | Wrong comment-format for .ll files. | |
generic/lib/gen_convert.py | ||
-32 | Wrong comment format for python files. The shebang also moved down from the first line, but it seems like the way we invoke this script as part of the build already picked a python executable, so it doesn't matter.. fwiw, this script can't be executed on its own anymore using ./gen_convert.py | |
generic/lib/integer/add_sat_impl.ll | ||
2 | Wrong comment format for .ll files. | |
generic/lib/integer/clz_if.ll | ||
2 | Wrong comment format for .ll files. | |
generic/lib/integer/clz_impl.ll | ||
2 | Wrong comment format for .ll files. | |
generic/lib/integer/sub_sat_impl.ll | ||
2 | Wrong comment format for .ll files. | |
generic/lib/shared/vstore_half_helpers.ll | ||
2 | Wrong comment format for .ll files. | |
generic/lib/subnormal_disable.ll | ||
2 | Wrong comment format for .ll files. | |
generic/lib/subnormal_use_default.ll | ||
2 | Wrong comment format for .ll files. Use a semi-colon at the start of the line. | |
ptx-nvidiacl/lib/SOURCES | ||
1 ↗ | (On Diff #86411) | build/metabuild.py errors out here... It treats all lines in the SOURCES files as entries and tries to make directories for anything it thinks needs a parent directory... |
ptx/lib/SOURCES | ||
1 | build/metabuild.py errors out here... It treats all lines in the SOURCES files as entries and tries to make directories for anything it thinks needs a parent directory... | |
ptx/lib/integer/add_sat.ll | ||
2 | Wrong comment format for .ll files. Use semi-colon at beginning of the line. | |
ptx/lib/integer/sub_sat.ll | ||
2 | Wrong comment format for .ll files | |
r600/lib/SOURCES | ||
1 ↗ | (On Diff #86411) | build/metabuild.py errors out here... It treats all lines in the SOURCES files as entries and tries to make directories for anything it thinks needs a parent directory... |
This isn't valid python syntax anymore. Prefix the license lines with '#'.