[PATCH] D85650: [llvm] support graceful failure of DataLayout parsing

Alex Zinenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 17 04:35:21 PDT 2020


ftynse updated this revision to Diff 285962.
ftynse added a comment.

Remove duplication between trySet* and set*, only keep the set* API that now
returns Error on failure instead of asserting. I haven't realized these were
private methods so there were no users in the wild that could be depending on
the current API or behavior.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85650

Files:
  llvm/include/llvm/IR/DataLayout.h
  llvm/lib/IR/DataLayout.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85650.285962.patch
Type: text/x-patch
Size: 18795 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200817/742f0b75/attachment.bin>


More information about the llvm-commits mailing list