[PATCH] D79570: [MC] Fix PR45805: infinite recursion in assembler

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 7 06:45:49 PDT 2020


thopre created this revision.
thopre added reviewers: MaskRay, jhenderson.
thopre added a project: LLVM.
Herald added a subscriber: hiraditya.
thopre updated this revision to Diff 262635.
thopre added a comment.

Default initialize InsertionSuccess


Give up folding an expression if one of the operands is being laid out.
This prevents hitting an infinite recursion when a fill size expression
refers to a later fragment since computing the offset of that fragment
would require laying out the fill fragment and thus computing its size
expression.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D79570

Files:
  llvm/include/llvm/MC/MCAsmLayout.h
  llvm/lib/MC/MCAssembler.cpp
  llvm/lib/MC/MCExpr.cpp
  llvm/test/MC/AsmParser/pr45805.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79570.262635.patch
Type: text/x-patch
Size: 3731 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200507/25a387cb/attachment.bin>


More information about the llvm-commits mailing list