Open zoybai opened 6 years ago
cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"tests/swap_mutex.h\" src/lockhammer.c -o build/lh_swap_mutex -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"../../ext/mysql/event_mutex.h\" src/lockhammer.c -o build/lh_event_mutex -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"../../ext/mysql/cas_event_mutex.h\" src/lockhammer.c -o build/lh_cas_event_mutex -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"tests/cas_lockref.h\" src/lockhammer.c -o build/lh_cas_lockref -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"tests/cas_rw_lock.h\" src/lockhammer.c -o build/lh_cas_rw_lock -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"tests/incdec_refcount.h\" src/lockhammer.c -o build/lh_incdec_refcount -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"../../ext/linux/ticket_spinlock.h\" src/lockhammer.c -o build/lh_ticket_spinlock -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"../../ext/linux/queued_spinlock.h\" src/lockhammer.c -o build/lh_queued_spinlock -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"tests/empty.h\" src/lockhammer.c -o build/lh_empty -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"../../ext/jvm/jvm_objectmonitor.h\" src/lockhammer.c -o build/lh_jvm_objectmonitor -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ In file included from src/lockhammer.c:48:0: ./../../ext/jvm/jvm_objectmonitor.h: In function ‘compute_abstime’: ./../../ext/jvm/jvm_objectmonitor.h:186:9: warning: unused variable ‘status’ [-Wunused-variable] int status = clock_gettime(CLOCK_MONOTONIC, &now); ^~~~~~ ./../../ext/jvm/jvm_objectmonitor.h: In function ‘unparkObjectWaiter’: ./../../ext/jvm/jvm_objectmonitor.h:300:9: warning: variable ‘status’ set but not used [-Wunused-but-set-variable] int status = pthread_mutex_lock(&node->_mutex); ^~~~~~ ./../../ext/jvm/jvm_objectmonitor.h: In function ‘jvmObjectMonitorExit’: ./../../ext/jvm/jvm_objectmonitor.h:720:13: warning: unused variable ‘QMode’ [-Wunused-variable] int QMode = Knob_QMode; ^~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DNDEBUG -DATOMIC_TEST=\"../../ext/tbb/tbb_spin_rw_mutex.h\" src/lockhammer.c -o build/lh_tbb_spin_rw_mutex -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"../../ext/linux/osq_lock.h\" src/lockhammer.c -o build/lh_osq_lock -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"../../ext/linux/hybrid_spinlock.h\" src/lockhammer.c -o build/lh_hybrid_spinlock -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ In file included from src/lockhammer.c:48:0: ./../../ext/linux/hybrid_spinlock.h: In function ‘hybrid_spinlock_slowpath’: ./../../ext/linux/hybrid_spinlock.h:127:11: warning: unused variable ‘tmp’ [-Wunused-variable] unsigned tmp, tmp2, tmp3; ^~~ ./../../ext/linux/hybrid_spinlock.h:96:6: warning: unused variable ‘new’ [-Wunused-variable] u32 new, old, tail, val, ticketval; ^~~ ./../../ext/linux/hybrid_spinlock.h: In function ‘lock_acquire’: ./../../ext/linux/hybrid_spinlock.h:188:11: warning: unused variable ‘tmp’ [-Wunused-variable] unsigned tmp, tmp2, tmp3; ^~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~ cc -Wall -g -O3 -I. -I./include -I../../ext/mysql/include -I../../ext/linux/include -I../../ext/tbb/include -DATOMIC_TEST=\"../../ext/linux/hybrid_spinlock_fastdequeue.h\" src/lockhammer.c -o build/lh_hybrid_spinlock_fastdequeue -lpthread -lm In file included from src/lockhammer.c:46:0: ./include/perf_timer.h: In function ‘evaluate_blackhole’: ./include/perf_timer.h:378:13: warning: unused variable ‘LOOP_TEST_OVERHEAD’ [-Wunused-variable] int64_t LOOP_TEST_OVERHEAD = evaluate_loop_overhead(NUMTRIES); ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:376:12: warning: unused variable ‘percent’ [-Wunused-variable] double percent; ^~~~~~~ ./include/perf_timer.h:375:13: warning: variable ‘elapsed_total_diff’ set but not used [-Wunused-but-set-variable] int64_t elapsed_total_diff; ^~~~~~~~~~~~~~~~~~ ./include/perf_timer.h:374:13: warning: variable ‘outer_inner_diff’ set but not used [-Wunused-but-set-variable] int64_t outer_inner_diff; ^~~~~~~~~~~~~~~~ In file included from src/lockhammer.c:48:0: ./../../ext/linux/hybrid_spinlock_fastdequeue.h: In function ‘hybrid_spinlock_slowpath’: ./../../ext/linux/hybrid_spinlock_fastdequeue.h:130:11: warning: unused variable ‘tmp’ [-Wunused-variable] unsigned tmp, tmp2, tmp3; ^~~ ./../../ext/linux/hybrid_spinlock_fastdequeue.h:99:6: warning: unused variable ‘new’ [-Wunused-variable] u32 new, old, tail, val, ticketval; ^~~ ./../../ext/linux/hybrid_spinlock_fastdequeue.h: In function ‘lock_acquire’: ./../../ext/linux/hybrid_spinlock_fastdequeue.h:221:11: warning: unused variable ‘tmp’ [-Wunused-variable] unsigned tmp, tmp2, tmp3; ^~~ src/lockhammer.c: In function ‘main’: src/lockhammer.c:101:14: warning: unused variable ‘buf’ [-Wunused-variable] char buf[128]; ^~~ src/lockhammer.c: In function ‘hmr’: src/lockhammer.c:349:12: warning: unused variable ‘tickspns’ [-Wunused-variable] double tickspns = x->tickspns; ^~~~~~~~
This and #20 appear to be duplicates.
All of the included lock algorithms should be built with -Wall enabled and any warnings encountered should be fixed.