Skip to content

Commit f2aa961

Browse files
committed
Update comments
1 parent 921b22d commit f2aa961

5 files changed

Lines changed: 7 additions & 7 deletions

File tree

src/tests/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ cfg_has_atomic_64! {
5252
}
5353
test_common!(i128);
5454
test_common!(u128);
55-
#[cfg(not(all(valgrind, target_arch = "powerpc64")))] // TODO: Hang (as of Valgrind 3.25)
55+
#[cfg(not(all(valgrind, target_arch = "powerpc64")))] // TODO(powerpc64): Hang (as of Valgrind 3.25)
5656
cfg_has_atomic_128! {
5757
test_atomic!(i128);
5858
test_atomic!(u128);

tests/no-std-qemu/.cargo/config.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ runner = "qemu-system-arm -M lm3s6965evb -cpu cortex-m4 -display none -semihosti
1010
[target.thumbv7em-none-eabihf]
1111
runner = "qemu-system-arm -M lm3s6965evb -cpu cortex-m4 -display none -semihosting -kernel"
1212
[target.thumbv8m.base-none-eabi]
13-
# TODO: As of QEMU 9.2, QEMU doesn't support -cpu cortex-m23
13+
# TODO: As of QEMU 10.1, QEMU doesn't support -cpu cortex-m23
1414
runner = "qemu-system-arm -M lm3s6965evb -cpu cortex-m33 -display none -semihosting -kernel"
1515
[target.thumbv8m.main-none-eabi]
1616
runner = "qemu-system-arm -M lm3s6965evb -cpu cortex-m33 -display none -semihosting -kernel"

tools/build.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ default_targets=(
7171

7272
# hexagon
7373
# rustc -Z unstable-options --print all-target-specs-json | jq -r '. | to_entries[] | if .value.arch == "hexagon" then .key else empty end'
74-
# TODO: error: symbol 'fma' is already defined
74+
# TODO(hexagon): error: symbol 'fma' is already defined
7575
# hexagon-unknown-linux-musl
7676

7777
# loongarch
@@ -308,7 +308,7 @@ build() {
308308
;;
309309
mips-*-linux-* | mipsel-*-linux-*)
310310
if ! grep -Eq "^${target}$" <<<"${rustup_target_list}"; then
311-
# TODO: LLVM bug: Undefined temporary symbol error when building std.
311+
# TODO(mips): LLVM bug: Undefined temporary symbol error when building std.
312312
target_rustflags+=" -C opt-level=1"
313313
fi
314314
;;

tools/no-std.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ run() {
166166
fi
167167
case "${target}" in
168168
xtensa*)
169-
# TODO: run test with simulator on CI
169+
# TODO(xtensa): run test with simulator on CI
170170
if ! type -P wokwi-server >/dev/null; then
171171
printf '%s\n' "no-std test for ${target} requires wokwi-server (switched to build-only)"
172172
subcmd=build

tools/test.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -256,7 +256,7 @@ run() {
256256
fi
257257

258258
case "${target}" in
259-
# TODO: segmentation fault (also happen without this crate)
259+
# TODO(hexagon): segmentation fault (also happen without this crate)
260260
hexagon-unknown-linux-musl) ;;
261261
*)
262262
# LTO + doctests is very slow on some platforms (probably related to the fact that they compile binaries for each example)
@@ -281,7 +281,7 @@ run() {
281281
case "${target}" in
282282
# cannot find rsbegin.o/rsend.o when building std: https://github.com/rust-lang/rust/issues/146739
283283
*-windows-gnu*) ;;
284-
# TODO: haw to pass build-std-features to cargo-careful?
284+
# TODO(hexagon): haw to pass build-std-features to cargo-careful?
285285
hexagon-unknown-linux-musl) ;;
286286
*)
287287
if [[ ${#build_std[@]} -gt 0 ]]; then

0 commit comments

Comments
 (0)