[PATCH] D155023: Don't assert on a non-pointer value being used for a "p" inline asm constraint.

[Amara Emerson via Phabricator via llvm-commits]

aemerson added a comment.

As I mentioned there is *already* code out in the wild that are relying on this to work. GCC also accepts this code, and so IMO it is the defacto semantics of this feature. In this case the IR semantics are wrong.

As it currently stands clang with asserts will crash on code that it previously compiled cleanly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155023