[llvm] r231270 - Make DataLayout Non-Optional in the Module

Mehdi Amini mehdi.amini at apple.com
Wed Mar 4 12:50:29 PST 2015


> On Mar 4, 2015, at 11:14 AM, Reid Kleckner <rnk at google.com> wrote:
> 
> This broke the Clang build. Hopefully r231278 fixes it. Please try to test Clang when making cross-cutting changes like this, it's very disruptive to the build bot infrastructure.

Sorry for the inconvenience, it was my intention.

I had a patch for Clang ready yesterday, before pushing the LLVM one, but when I wanted to push in clang, I had to rebase and the code was not compiling anymore I had a compile failure in Clang. And by the time I got my llvm and clang in sync again, and by the time to run the test suite, you got the exact same patch in.

Best,

Mehdi







More information about the llvm-commits mailing list