aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/Frontend/CompilerInvocation.cpp
diff options
context:
space:
mode:
authorNick Desaulniers <nickdesaulniers@users.noreply.github.com>2023-12-04 08:15:32 -0800
committerGitHub <noreply@github.com>2023-12-04 08:15:32 -0800
commitec000a65553bac55a1f4b8b21c365d48dc6a540a (patch)
tree005f3df72c2a3566509a2e8b660e822b45bc9ccd /clang/lib/Frontend/CompilerInvocation.cpp
parent47a40e80c553b07a08f0f17fd68c12ae9b9a2a67 (diff)
downloadllvm-ec000a65553bac55a1f4b8b21c365d48dc6a540a.zip
llvm-ec000a65553bac55a1f4b8b21c365d48dc6a540a.tar.gz
llvm-ec000a65553bac55a1f4b8b21c365d48dc6a540a.tar.bz2
[TargetInstrInfo] update INLINEASM memoperands once (#74135)
In commit b05335989239 ("[X86InstrInfo] support memfold on spillable inline asm (#70832)"), I had a last minute fix to update the memoperands. I originally did this in the parent foldInlineAsmMemOperand call, updated the mir test via update_mir_test_checks.py, but then decided to move it to the child call of foldInlineAsmMemOperand. But I forgot to rerun update_mir_test_checks.py. That last minute change caused the same memoperand to be added twice when recursion occurred (for tied operands). I happened to get lucky that trailing content omitted from the CHECK line doesn't result in test failure. But rerunning update_mir_test_checks.py on the mir test added in that commit produces updated output. This is resulting in updates to the test that: 1. conflate additions to the test in child commits with simply updating the test as it should have been when first committed. 2. look wrong because the same memoperand is specified twice (we don't deduplicate memoperands when added). Example: INLINEASM ... :: (load (s32) from %stack.0) (load (s32) from %stack.0) Fix the bug, so that in child commits, we don't have additional unrelated test changes (which would be wrong anyways) from simply running update_mir_test_checks.py. Link: #20571
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions