[PATCH] D133202: [Clang] Avoid __builtin_assume_aligned crash when the 1st arg is array type

[Lin Yurong via Phabricator via cfe-commits]

yronglin added a comment.

Thanks very much for your comments @rjmccall @rsmith , I've take a look at D45383 <https://reviews.llvm.org/D45383>, I believe that user code isn't allowed to declare  __builtin_*, but seems `intrin0.inl.h` is a system header on windows, should we keep compatibility(like `__va_start` in D45383 <https://reviews.llvm.org/D45383>) with it? or breaking it, but I'd like some further guidance. can you help me make that decision? I think you guys are experts in this area and have more experience than me, I'll see if I can accomplish soon! (maybe I should submit a new RFC patch).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133202