Missing cases in switch statements in drx-scattergather-aarch64.cpp

[egrimley-arm]

Clang warns about these missing cases. The switch statements are at:

dynamorio/suite/tests/client-interface/drx-scattergather-aarch64.cpp

Line 2442 in eb5bd83

switch (element_size_) {

dynamorio/suite/tests/client-interface/drx-scattergather-aarch64.cpp

Line 2446 in eb5bd83

switch (stored_value_size_) {

When I added default: assert(false); some tests failed so perhaps falling through is correct, but someone should check before adding default: break;.

I think @jackgallagher-arm is the expert on this file.

[egrimley-arm]

There's also a missing case in the switch statement at

dynamorio/clients/drcachesim/tests/burst_syscall_inject.cpp

Line 472 in eb5bd83

switch (memref.marker.marker_type) {

which could be dealt with at the same time. @abhinav92003 seems to be the author of the surrounding lines there.