Fix 11 clang-tidy checks

[bcaddy]

Clang-Tidy Checks

Fixed and enabled the following clang-tidy Checks

• performance-unnecessary-value-param check
• performance-inefficient-vector-operation check
• modernize-use-auto and its alias hicpp-use-auto.
• performance-for-range-copy check
• performance-faster-string-find check
• modernize-use-default-member-init check
• modernize-loop-convert tidy check
• readability-const-return-type
• readability-simplify-boolean-expr
• google-build-namespaces (and its alias cert-dcl59-cpp)

Disabled cert-err58-cpp. It provides minimal benifit to us and flags every single gtest TEST macro as failing.

Other

• Removed unsafe pointer alias to a DeviceVector pointer in io.cpp
• Append make type to clang-tidy log files

[alwinm]

It looks good to me