[PATCH] D131979: [clang][UBSan] Fix __builtin_assume_aligned crash

Wang Yihan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 29 10:52:14 PDT 2022


yihanaa added a comment.

In D131979#3753706 <https://reviews.llvm.org/D131979#3753706>, @yihanaa wrote:

> Keep same behavior with current clang, don't emit an error when 1st arg is volatile qualified.
>
> Ignore the implicit cast from user-written-type to 'const void *' on the 1st arg, because CodeGen need user-written-type to generate correct TypeDescriptor (this class in compiler-rt/UBSan), then, clang will emit cast instruction which cast type from user-written-type to VoidPtr in CodeGen.
>
> But I still have some questions, for example: if someone pass a array type(e.g. char[1]) as the 1st arg, what should the correct TypeDescriptor? char[1] OR char *, clang will generate char * in this patch, you are experts and what do you all think?

Hi John, can you please look at here, this is current version of patch's comment, and probably folded by Phabricator because of too many comments. Thanks!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131979/new/

https://reviews.llvm.org/D131979



More information about the cfe-commits mailing list