[PATCH] D145416: [clang] model 'p' inline asm constraint as reading memory

[Nick Desaulniers via Phabricator via cfe-commits]

nickdesaulniers added inline comments.


================
Comment at: clang/lib/Basic/TargetInfo.cpp:747-748
       break;
     case 'g': // general register, memory operand or immediate integer.
     case 'X': // any operand.
       Info.setAllowsRegister();
----------------
nickdesaulniers wrote:
> I wonder if we should make this change for outputs, too?
1. https://reviews.llvm.org/D145429
2. https://reviews.llvm.org/D145430


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145416