search for: bundle_align_start

Maybe I'm jumping the gun, but some initial thoughts after browsing the MC layer changes: * .bundle_align_start -> .bundle_align_mode N ? * .bundle_align_end -> .bundle_align_mode 0 ? * Add unit test showing .bundle_lock/unlock can be nested. I like the way you modified the existing ARM tests to show how the existing instructions are predicated. Overall, I think the concept of Native Client is wel...

On Wed, Dec 5, 2012 at 6:01 PM, Greg Fitzgerald <garious at gmail.com> wrote: >> But if you just want to see the code and the diff against upstream... > > If others are interested as well, here's all your changes via Github's > "compare" feature: > > https://github.com/garious/llvm/compare/master...chromium > > Fascinating stuff guys. What a

On Thu, Dec 6, 2012 at 11:59 AM, Greg Fitzgerald <garious at gmail.com> wrote: > Maybe I'm jumping the gun, but some initial thoughts after browsing the MC > layer changes: > Greg, it's great to have early comments on this. > * .bundle_align_start -> .bundle_align_mode N ? > * .bundle_align_end -> .bundle_align_mode 0 ? The initial proposal does not cover .bundle_align{start|end} on purpose, to keep things simple. We do plan to add them, eventually, probably as attributes on .bundle_lock. > * Add unit test showing .bundle_lo...

...On Thu, Dec 6, 2012 at 11:59 AM, Greg Fitzgerald <garious at gmail.com> > wrote: > > Maybe I'm jumping the gun, but some initial thoughts after browsing the > MC > > layer changes: > > > > Greg, it's great to have early comments on this. > > > * .bundle_align_start -> .bundle_align_mode N ? > > * .bundle_align_end -> .bundle_align_mode 0 ? > > The initial proposal does not cover .bundle_align{start|end} on > purpose, to keep things simple. We do plan to add them, eventually, > probably as attributes on .bundle_lock. > > > *...