[llvm-commits] PATCH: Aligned bundling support in LLVM MC

Jim Grosbach grosbach at apple.com
Wed Dec 19 16:14:07 PST 2012


Hi Eli,

The code all looks fine to me. Pretty excellent, in fact. I'm OK with this going in. If we change our minds (see below question), we can always back out.

I still have a bit of a funny feeling about doing this sort of thing in the assembler at all, though. It really seems it should be done as part of the target back end using alignment directives in the code stream. That ship may have already sailed, though, if we want to keep compatibility w/ GNU's binutils on these sorts of features. Chris, do you have an opinion on the general direction here?

Thanks,
  Jim

On Dec 14, 2012, at 3:09 PM, Eli Bendersky <eliben at google.com> wrote:

> Hello,
> 
> Following the discussion here:
> http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-December/056754.html
> 
> I'm sending the first set of patches to implement ailgned bundling
> support in LLVM MC. The proposal and implementation are documented
> here:
> 
> https://sites.google.com/a/chromium.org/dev/nativeclient/pnacl/aligned-bundling-support-in-llvm
> 
> There are two patches: one with the implementation and another with the tests.
> 
> Please take a look,
> Eli
> <mc-bundling.impl.1.patch><mc-bundling.tests.1.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list