onnxruntime icon indicating copy to clipboard operation
onnxruntime copied to clipboard
verified publisher icon microsoft

Fix prefast warnings

Open tianleiwu opened this issue 3 years ago • 4 comments

Description:

Fix prefast warnings:

  • C26451 (in onnxruntime/contrib_ops/cpu/transformers/generation_device_helper.cc) (in onnxruntime/contrib_ops/cpu/transformers/greedy_search_impl_gpt.h) (in onnxruntime/contrib_ops/cpu/bert/attention_cpu_base.h) (in onnxruntime/contrib_ops/cpu/bert/attention_helper.h) (in onnxruntime/contrib_ops/cpu/transformers/subgraph_t5_decoder.cc) (in onnxruntime/core/providers/cuda/generator/range.cc) (in onnxruntime/contrib_ops/cuda/transformers/generation_device_helper.cc) (in onnxruntime/contrib_ops/cuda/bert/longformer_attention.cc) (in onnxruntime/contrib_ops/cuda/bert/attention.cc)

  • C26436 (in onnxruntime/contrib_ops/cpu/transformers/beam_search_impl_base.h) (in onnxruntime/contrib_ops/cpu/transformers/greedy_search_impl_base.h)

  • C26814 (in onnxruntime/contrib_ops/cpu/transformers/generation_device_helper.cc)

Motivation and Context

  • Why is this change required? What problem does it solve?
  • If it fixes an open issue, please link to the issue here.

tianleiwu avatar Sep 19 '22 22:09 tianleiwu

This pull request fixes 2 alerts when merging 436d0a184e90d51b810da247bfde5406cdd50c94 into 14eb3cf48516c1161024c382b370e380f205cf25 - view on LGTM.com

fixed alerts:

  • 2 for Multiplication result converted to larger type

lgtm-com[bot] avatar Sep 20 '22 00:09 lgtm-com[bot]

This pull request fixes 2 alerts when merging 824c66e3f619b27802d4a42fb803f5fb4292268a into 189aef2beaecdbdd3a68fbf569d9891c11d9777c - view on LGTM.com

fixed alerts:

  • 2 for Multiplication result converted to larger type

lgtm-com[bot] avatar Sep 20 '22 19:09 lgtm-com[bot]

This pull request fixes 2 alerts when merging 58586d866fbc25c59be6e35fc57f8e22779ed00c into 078ceab1dba970c1598548c01642e6e2517d9884 - view on LGTM.com

fixed alerts:

  • 2 for Multiplication result converted to larger type

lgtm-com[bot] avatar Sep 22 '22 23:09 lgtm-com[bot]

This pull request fixes 2 alerts when merging 4edcf3d7846e3b749a7dd8cc490771cf570a67f8 into a24b41d92e2aaa51087f52dbf57d2d7f6e7dfd82 - view on LGTM.com

fixed alerts:

  • 2 for Multiplication result converted to larger type

lgtm-com[bot] avatar Sep 23 '22 07:09 lgtm-com[bot]

This pull request fixes 2 alerts when merging 4a0729ffab7460d459c2fcc7b2f8f030e067a952 into 2ef1f8b93ea7a3717d7e95d51963947364f003f7 - view on LGTM.com

fixed alerts:

  • 2 for Multiplication result converted to larger type

lgtm-com[bot] avatar Sep 23 '22 19:09 lgtm-com[bot]