Add DWARFFormValue::getAsBlockData(), getAsBlockSize() and getAsFlag().
Frédéric Riss
friss at apple.com
Wed Sep 3 11:56:43 PDT 2014
> On 03 Sep 2014, at 20:52, David Blaikie <dblaikie at gmail.com> wrote:
>
> What's the need for the getAsFlag function?
FC_Flag is a separate class. I just mimicked the other variants. I can however fold FC_Flag into getAsUnsignedConstant(). This would make the followup patch about symbolic constants a tiny bit simpler.
> Perhaps the getAsBlockData/getAsBlockSize could be one function, returning an Optional<ArrayRef<uint8_t*>> ? ("getAsBlock”?)
Great suggestion. I’ll do that and rebase the rest.
Fred
>
> On Wed, Sep 3, 2014 at 11:47 AM, Frédéric Riss <friss at apple.com <mailto:friss at apple.com>> wrote:
> These new accessors will be used in subsequent patches.
>
> OK to commit?
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140903/891eca3a/attachment.html>
More information about the llvm-commits
mailing list