[PATCH 1/3] Add missing case statements to LLParser for improved error messages

Rafael EspĂ­ndola rafael.espindola at gmail.com
Thu May 2 11:02:13 PDT 2013


On 18 April 2013 09:29, Stephen Lin <swlin at post.harvard.edu> wrote:
> Seems like someone else had the same idea I did. I've updated the
> patch to reflect the latest changes to trunk.
>
> The 'sspstrong' and 'noduplicate' issues seem to be fixed already;
> I've standardized the code organization to avoid splitting the set of
> function attributes into two columns, since that's what is done now in
> trunk in one function (but not another) and moved 'align' to the
> correct category again.

Having a single column does look better. Can you include a testcase
with the new error message?

Thanks,
Rafael



More information about the llvm-commits mailing list