<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif;font-size:small">Fixed the name to BitcodeReader::parseAlignmentValue, added test (test/Bitcode/Inputs/invalid-align.bc), and updated test/BitcodeInputs/invalid.tests.</div><div class="gmail_default" style="font-family:tahoma,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;font-size:small">jfb: Could you please commit this since I do not have commit privilege yet. Rafael has reviewed it and has given it a LGTM.</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">Karl Schimpf <br></div></div>
<br><div class="gmail_quote">On Thu, Feb 19, 2015 at 7:09 PM, Filipe Cabecinhas <span dir="ltr"><<a href="mailto:filcab@gmail.com" target="_blank">filcab@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Feel free to use test/Bitcode/invalid.test and add a test/Bitcode/Inputs/invalid-*.bc, since it's an invalid bitcode file. I've been putting them there so they don't “pollute” test/Bitcode.<div><br></div><div>Thanks,<br><div class="gmail_extra"><br clear="all"><div><div>  Filipe<br></div></div><div><div class="h5">
<br><div class="gmail_quote">On Thu, Feb 19, 2015 at 6:11 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It is OK to check in a .bc file.<br>
<br>
BitcodeReader::ParseAlignmentValue<br>
<br>
Please use the new convention (start functions with lowercase).<br>
<br>
LGTM with that.<br>
</blockquote></div><br></div></div></div></div></div>
</blockquote></div><br></div>