Page MenuHomePhabricator
Differential D145430

[clang] accept inline asm 'p' constraint on outputs
Needs ReviewPublic
Actions

Authored by nickdesaulniers on Mar 6 2023, 2:34 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

GCC allows this; we reject it.

Link: https://reviews.llvm.org/D145416#inline-1404524

Diff Detail

Event Timeline

nickdesaulniers created this revision.Mar 6 2023, 2:34 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 6 2023, 2:34 PM
nickdesaulniers requested review of this revision.Mar 6 2023, 2:34 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 6 2023, 2:34 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
nickdesaulniers added a parent revision: D145429: add test case.Mar 6 2023, 2:34 PM
nickdesaulniers mentioned this in D145416: [clang] model 'p' inline asm constraint as reading memory.
nickdesaulniers updated this revision to Diff 502822.Mar 6 2023, 3:01 PM
  • rebase, additional unit test
Harbormaster completed remote builds in B217723: Diff 502822.Mar 6 2023, 8:07 PM

Revision Contents
Changeset List

PathSize
clang/
lib/
Basic/
1 line
test/
Sema/
6 lines

Diff 502822

View Options

clang/lib/Basic/TargetInfo.cpp

Loading...
View Options

clang/test/Sema/inline-asm-validate.c

Loading...