[PATCH] D94433: [ValueTracking] Check that alignment is non-zero in computeKnownBitsFromAssume (PR48713).

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

Any updates on changing clang to avoid this? 
There's a recent comment on:
https://llvm.org/PR48713
...that suggests fuzzers have found their way to this crash too.

Wouldn't we still want this check in the optimizer given the text in LangRef? 
"if the alignment is set to zero, the alignment of the function is set by the target to whatever it feels convenient":
https://llvm.org/docs/LangRef.html#global-variables


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94433