[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 llvm-commits at lists.llvm.org
Wed Jul 12 08:01:43 PDT 2023


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



More information about the llvm-commits mailing list